[llvm] 167b598 - [ConstraintElim] Remove redundant debug output (NFC).

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Sat Nov 11 05:01:29 PST 2023


Author: Florian Hahn
Date: 2023-11-11T13:01:12Z
New Revision: 167b598648dfcee2f23426bfa972e53f8ac16722

URL: https://github.com/llvm/llvm-project/commit/167b598648dfcee2f23426bfa972e53f8ac16722
DIFF: https://github.com/llvm/llvm-project/commit/167b598648dfcee2f23426bfa972e53f8ac16722.diff

LOG: [ConstraintElim] Remove redundant debug output (NFC).

The removed code was printing `Processing facts ...` a second time.

Added: 
    

Modified: 
    llvm/lib/Transforms/Scalar/ConstraintElimination.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/Scalar/ConstraintElimination.cpp b/llvm/lib/Transforms/Scalar/ConstraintElimination.cpp
index 368082b0150a827..8ec2ee47db1b70c 100644
--- a/llvm/lib/Transforms/Scalar/ConstraintElimination.cpp
+++ b/llvm/lib/Transforms/Scalar/ConstraintElimination.cpp
@@ -1599,7 +1599,7 @@ static bool eliminateConstraints(Function &F, DominatorTree &DT, LoopInfo &LI,
       LLVM_DEBUG(dbgs() << "fact to add to the system: "
                         << CmpInst::getPredicateName(Pred) << " ";
                  A->printAsOperand(dbgs()); dbgs() << ", ";
-                 B->printAsOperand(dbgs()); dbgs() << "\n");
+                 B->printAsOperand(dbgs(), false); dbgs() << "\n");
       if (Info.getCS(CmpInst::isSigned(Pred)).size() > MaxRows) {
         LLVM_DEBUG(
             dbgs()
@@ -1607,14 +1607,6 @@ static bool eliminateConstraints(Function &F, DominatorTree &DT, LoopInfo &LI,
         return;
       }
 
-      LLVM_DEBUG({
-        dbgs() << "Processing fact to add to the system: " << Pred << " ";
-        A->printAsOperand(dbgs());
-        dbgs() << ", ";
-        B->printAsOperand(dbgs(), false);
-        dbgs() << "\n";
-      });
-
       Info.addFact(Pred, A, B, CB.NumIn, CB.NumOut, DFSInStack);
       if (ReproducerModule && DFSInStack.size() > ReproducerCondStack.size())
         ReproducerCondStack.emplace_back(Pred, A, B);


        


More information about the llvm-commits mailing list