[ConstriantElimination] Add NODEBUG condition around dump

Zain Jaffal · Zain Jaffal · commit df2ea2fc2839 · 2023-02-15T18:53:22.000Z
diff --git a/llvm/lib/Analysis/ConstraintSystem.cpp b/llvm/lib/Analysis/ConstraintSystem.cpp
@@ -114,6 +114,7 @@ bool ConstraintSystem::mayHaveSolutionImpl() {
 
 SmallVector<std::string> ConstraintSystem::getVarNamesList() const {
   SmallVector<std::string> Names(Value2Index.size(), "");
+#ifndef NDEBUG
   for (auto &[V, Index] : Value2Index) {
     std::string OperandName;
     if (V->getName().empty())
@@ -122,10 +123,12 @@ SmallVector<std::string> ConstraintSystem::getVarNamesList() const {
       OperandName = std::string("%") + V->getName().str();
     Names[Index - 1] = OperandName;
   }
+#endif
   return Names;
 }
 
 void ConstraintSystem::dump() const {
+#ifndef NDEBUG
   if (Constraints.empty())
     return;
   SmallVector<std::string> Names = getVarNamesList();
@@ -143,6 +146,7 @@ void ConstraintSystem::dump() const {
     LLVM_DEBUG(dbgs() << join(Parts, std::string(" + "))
                       << " <= " << std::to_string(Row[0]) << "\n");
   }
+#endif
 }
 
 bool ConstraintSystem::mayHaveSolution() {

Original file line number	Diff line number	Diff line change
`@@ -114,6 +114,7 @@ bool ConstraintSystem::mayHaveSolutionImpl() {`
`114`	`114`
`115`	`115`	`SmallVector<std::string> ConstraintSystem::getVarNamesList() const {`
`116`	`116`	`SmallVector<std::string> Names(Value2Index.size(), "");`
	`117`	`+#ifndef NDEBUG`
`117`	`118`	`for (auto &[V, Index] : Value2Index) {`
`118`	`119`	`std::string OperandName;`
`119`	`120`	`if (V->getName().empty())`
`@@ -122,10 +123,12 @@ SmallVector<std::string> ConstraintSystem::getVarNamesList() const {`
`122`	`123`	`OperandName = std::string("%") + V->getName().str();`
`123`	`124`	`Names[Index - 1] = OperandName;`
`124`	`125`	`}`
	`126`	`+#endif`
`125`	`127`	`return Names;`
`126`	`128`	`}`
`127`	`129`
`128`	`130`	`void ConstraintSystem::dump() const {`
	`131`	`+#ifndef NDEBUG`
`129`	`132`	`if (Constraints.empty())`
`130`	`133`	`return;`
`131`	`134`	`SmallVector<std::string> Names = getVarNamesList();`
`@@ -143,6 +146,7 @@ void ConstraintSystem::dump() const {`
`143`	`146`	`LLVM_DEBUG(dbgs() << join(Parts, std::string(" + "))`
`144`	`147`	`<< " <= " << std::to_string(Row[0]) << "\n");`
`145`	`148`	`}`
	`149`	`+#endif`
`146`	`150`	`}`
`147`	`151`
`148`	`152`	`bool ConstraintSystem::mayHaveSolution() {`