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

Devang Patel dpatel at apple.com
Thu Jun 7 11:45:26 PDT 2007



Changes in directory llvm/lib/Transforms/Scalar:

GCSE.cpp updated: 1.60 -> 1.61
---
Log message:

Use DominatorTree instead of ETForest.



---
Diffs of the changes:  (+1 -3)

 GCSE.cpp |    4 +---
 1 files changed, 1 insertion(+), 3 deletions(-)


Index: llvm/lib/Transforms/Scalar/GCSE.cpp
diff -u llvm/lib/Transforms/Scalar/GCSE.cpp:1.60 llvm/lib/Transforms/Scalar/GCSE.cpp:1.61
--- llvm/lib/Transforms/Scalar/GCSE.cpp:1.60	Thu Jun  7 13:40:55 2007
+++ llvm/lib/Transforms/Scalar/GCSE.cpp	Thu Jun  7 13:45:06 2007
@@ -48,7 +48,6 @@
     // This transformation requires dominator and immediate dominator info
     virtual void getAnalysisUsage(AnalysisUsage &AU) const {
       AU.setPreservesCFG();
-      AU.addRequired<ETForest>();
       AU.addRequired<DominatorTree>();
       AU.addRequired<ValueNumbering>();
     }
@@ -69,7 +68,6 @@
 
   // Get pointers to the analysis results that we will be using...
   DominatorTree &DT = getAnalysis<DominatorTree>();
-  ETForest &ET = getAnalysis<ETForest>();
   ValueNumbering &VN = getAnalysis<ValueNumbering>();
 
   std::vector<Value*> EqualValues;
@@ -145,7 +143,7 @@
             if (OtherI->getParent() == BB)
               Dominates = BlockInsts.count(OtherI);
             else
-              Dominates = ET.dominates(OtherI->getParent(), BB);
+              Dominates = DT.dominates(OtherI->getParent(), BB);
 
             if (Dominates) {
               // Okay, we found an instruction with the same value as this one






More information about the llvm-commits mailing list