Use DEBUG() instead of passing *DOUT to WriteAsOperand,
authorDan Gohman <gohman@apple.com>
Thu, 19 Feb 2009 19:32:06 +0000 (19:32 +0000)
committerDan Gohman <gohman@apple.com>
Thu, 19 Feb 2009 19:32:06 +0000 (19:32 +0000)
since the latter just passes a null reference when
debugging is not enabled.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@65060 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Transforms/Scalar/LoopStrengthReduce.cpp

index d906ee727c5da0861a68d40e193a72e2ab44bc99..d77025467c64fec1c007256916dbd7bb64467d00 100644 (file)
@@ -762,11 +762,9 @@ void BasedUser::RewriteInstructionToUseNewBase(const SCEVHandle &NewBase,
     // Replace the use of the operand Value with the new Phi we just created.
     Inst->replaceUsesOfWith(OperandValToReplace, NewVal);
 
-#ifndef NDEBUG
     DOUT << "      Replacing with ";
-    WriteAsOperand(*DOUT, NewVal, /*PrintType=*/false);
+    DEBUG(WriteAsOperand(*DOUT, NewVal, /*PrintType=*/false));
     DOUT << ", which has value " << *NewBase << " plus IMM " << *Imm << "\n";
-#endif
     return;
   }
 
@@ -830,11 +828,9 @@ void BasedUser::RewriteInstructionToUseNewBase(const SCEVHandle &NewBase,
                                               PN->getType());
         }
 
-#ifndef NDEBUG
         DOUT << "      Changing PHI use to ";
-        WriteAsOperand(*DOUT, Code, /*PrintType=*/false);
+        DEBUG(WriteAsOperand(*DOUT, Code, /*PrintType=*/false));
         DOUT << ", which has value " << *NewBase << " plus IMM " << *Imm << "\n";
-#endif
       }
 
       // Replace the use of the operand Value with the new Phi we just created.
@@ -1555,13 +1551,11 @@ void LoopStrengthReduce::StrengthReduceStridedIVUsers(const SCEVHandle &Stride,
     // Remember this in case a later stride is multiple of this.
     IVsByStride[Stride].addIV(Stride, CommonExprs, NewPHI, IncV);
 
-#ifndef NDEBUG
     DOUT << "  Inserted new PHI: IV=";
-    WriteAsOperand(*DOUT, NewPHI, /*PrintType=*/false);
+    DEBUG(WriteAsOperand(*DOUT, NewPHI, /*PrintType=*/false));
     DOUT << ", INC=";
-    WriteAsOperand(*DOUT, IncV, /*PrintType=*/false);
+    DEBUG(WriteAsOperand(*DOUT, IncV, /*PrintType=*/false));
     DOUT << "\n";
-#endif
   } else {
     DOUT << "  Rewriting in terms of existing IV of STRIDE " << *ReuseIV.Stride
          << " and BASE " << *ReuseIV.Base << "\n";
@@ -1618,11 +1612,9 @@ void LoopStrengthReduce::StrengthReduceStridedIVUsers(const SCEVHandle &Stride,
     // Emit the code for Base into the preheader.
     Value *BaseV = PreheaderRewriter.expandCodeFor(Base, PreInsertPt);
 
-#ifndef NDEBUG
     DOUT << "  Examining uses with BASE ";
-    WriteAsOperand(*DOUT, BaseV, /*PrintType=*/false);
+    DEBUG(WriteAsOperand(*DOUT, BaseV, /*PrintType=*/false));
     DOUT << ":\n";
-#endif
 
     // If BaseV is a constant other than 0, make sure that it gets inserted into
     // the preheader, instead of being forward substituted into the uses.  We do
@@ -1644,12 +1636,10 @@ void LoopStrengthReduce::StrengthReduceStridedIVUsers(const SCEVHandle &Stride,
       // FIXME: Use emitted users to emit other users.
       BasedUser &User = UsersToProcess.back();
 
-#ifndef NDEBUG
       DOUT << "    Examining use ";
-      WriteAsOperand(*DOUT, UsersToProcess.back().OperandValToReplace,
-                     /*PrintType=*/false);
+      DEBUG(WriteAsOperand(*DOUT, UsersToProcess.back().OperandValToReplace,
+                           /*PrintType=*/false));
       DOUT << " in Inst: " << *Inst;
-#endif
 
       // If this instruction wants to use the post-incremented value, move it
       // after the post-inc and use its value instead of the PHI.