[llvm-commits] [llvm] r108850 - /llvm/trunk/lib/Analysis/ScalarEvolutionNormalization.cpp

Dan Gohman gohman at apple.com
Tue Jul 20 09:34:50 PDT 2010


Author: djg
Date: Tue Jul 20 11:34:50 2010
New Revision: 108850

URL: http://llvm.org/viewvc/llvm-project?rev=108850&view=rev
Log:
Change an argument from an Instruction* to a Value*, which is all
that is needed here.

Modified:
    llvm/trunk/lib/Analysis/ScalarEvolutionNormalization.cpp

Modified: llvm/trunk/lib/Analysis/ScalarEvolutionNormalization.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/ScalarEvolutionNormalization.cpp?rev=108850&r1=108849&r2=108850&view=diff
==============================================================================
--- llvm/trunk/lib/Analysis/ScalarEvolutionNormalization.cpp (original)
+++ llvm/trunk/lib/Analysis/ScalarEvolutionNormalization.cpp Tue Jul 20 11:34:50 2010
@@ -26,7 +26,7 @@
 /// post-inc value when we cannot) or it can end up adding extra live-ranges to
 /// the loop, resulting in reg-reg copies (if we use the pre-inc value when we
 /// should use the post-inc value).
-static bool IVUseShouldUsePostIncValue(Instruction *User, Instruction *IV,
+static bool IVUseShouldUsePostIncValue(Instruction *User, Value *Operand,
                                        const Loop *L, DominatorTree *DT) {
   // If the user is in the loop, use the preinc value.
   if (L->contains(User)) return false;
@@ -47,15 +47,15 @@
   PHINode *PN = dyn_cast<PHINode>(User);
   if (!PN) return false;  // not a phi, not dominated by latch block.
 
-  // Look at all of the uses of IV by the PHI node.  If any use corresponds to
-  // a block that is not dominated by the latch block, give up and use the
+  // Look at all of the uses of Operand by the PHI node.  If any use corresponds
+  // to a block that is not dominated by the latch block, give up and use the
   // preincremented value.
   for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i)
-    if (PN->getIncomingValue(i) == IV &&
+    if (PN->getIncomingValue(i) == Operand &&
         !DT->dominates(LatchBlock, PN->getIncomingBlock(i)))
       return false;
 
-  // Okay, all uses of IV by PN are in predecessor blocks that really are
+  // Okay, all uses of Operand by PN are in predecessor blocks that really are
   // dominated by the latch block.  Use the post-incremented value.
   return true;
 }





More information about the llvm-commits mailing list