[llvm-commits] CVS: llvm/lib/Transforms/Scalar/IndVarSimplify.cpp LoopStrengthReduce.cpp

[Dan Gohman]

Changes in directory llvm/lib/Transforms/Scalar:

IndVarSimplify.cpp updated: 1.122 -> 1.123
LoopStrengthReduce.cpp updated: 1.141 -> 1.142
---
Log message:

Rename ScalarEvolution::deleteInstructionFromRecords to
deleteValueFromRecords and loosen the types to all it to accept
Value* instead of just Instruction*, since this is what
ScalarEvolution uses internally anyway. This allows more flexibility
for future uses.


---
Diffs of the changes:  (+5 -5)

 IndVarSimplify.cpp     |    6 +++---
 LoopStrengthReduce.cpp |    4 ++--
 2 files changed, 5 insertions(+), 5 deletions(-)


Index: llvm/lib/Transforms/Scalar/IndVarSimplify.cpp
diff -u llvm/lib/Transforms/Scalar/IndVarSimplify.cpp:1.122 llvm/lib/Transforms/Scalar/IndVarSimplify.cpp:1.123
--- llvm/lib/Transforms/Scalar/IndVarSimplify.cpp:1.122	Fri Jun 15 13:00:55 2007
+++ llvm/lib/Transforms/Scalar/IndVarSimplify.cpp	Tue Jun 19 09:28:31 2007
@@ -115,7 +115,7 @@
       for (unsigned i = 0, e = I->getNumOperands(); i != e; ++i)
         if (Instruction *U = dyn_cast<Instruction>(I->getOperand(i)))
           Insts.insert(U);
-      SE->deleteInstructionFromRecords(I);
+      SE->deleteValueFromRecords(I);
       DOUT << "INDVARS: Deleting: " << *I;
       I->eraseFromParent();
       Changed = true;
@@ -181,7 +181,7 @@
               GetElementPtrInst *NGEPI = new GetElementPtrInst(
                   NCE, Constant::getNullValue(Type::Int32Ty), NewAdd, 
                   GEPI->getName(), GEPI);
-              SE->deleteInstructionFromRecords(GEPI);
+              SE->deleteValueFromRecords(GEPI);
               GEPI->replaceAllUsesWith(NGEPI);
               GEPI->eraseFromParent();
               GEPI = NGEPI;
@@ -398,7 +398,7 @@
         // the PHI entirely.  This is safe, because the NewVal won't be variant
         // in the loop, so we don't need an LCSSA phi node anymore.
         if (NumPreds == 1) {
-          SE->deleteInstructionFromRecords(PN);
+          SE->deleteValueFromRecords(PN);
           PN->replaceAllUsesWith(ExitVal);
           PN->eraseFromParent();
           break;


Index: llvm/lib/Transforms/Scalar/LoopStrengthReduce.cpp
diff -u llvm/lib/Transforms/Scalar/LoopStrengthReduce.cpp:1.141 llvm/lib/Transforms/Scalar/LoopStrengthReduce.cpp:1.142
--- llvm/lib/Transforms/Scalar/LoopStrengthReduce.cpp:1.141	Fri Jun 15 09:38:12 2007
+++ llvm/lib/Transforms/Scalar/LoopStrengthReduce.cpp	Tue Jun 19 09:28:31 2007
@@ -226,7 +226,7 @@
       for (unsigned i = 0, e = I->getNumOperands(); i != e; ++i)
         if (Instruction *U = dyn_cast<Instruction>(I->getOperand(i)))
           Insts.insert(U);
-      SE->deleteInstructionFromRecords(I);
+      SE->deleteValueFromRecords(I);
       I->eraseFromParent();
       Changed = true;
     }
@@ -1488,7 +1488,7 @@
             DeadInsts.insert(BO);
             // Break the cycle, then delete the PHI.
             PN->replaceAllUsesWith(UndefValue::get(PN->getType()));
-            SE->deleteInstructionFromRecords(PN);
+            SE->deleteValueFromRecords(PN);
             PN->eraseFromParent();
           }
         }