Changed insert instructions only for CONSTANTS - back to original

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

diff --git a/lib/Transforms/HoistPHIConstants.cpp b/lib/Transforms/HoistPHIConstants.cpp
index 6e903498f57ee5a7a528d9eb578cd24a573aa17b..d08deba7781440ada93dbd9569cf669ccd88a635 100644 (file)
--- a/lib/Transforms/HoistPHIConstants.cpp
+++ b/lib/Transforms/HoistPHIConstants.cpp
@@ -30,12 +30,12 @@ static Value *NormalizePhiOperand(PHINode *PN, Value *CPV,
   // Check if we've already inserted a copy for this constant in Pred
   // Note that `copyCache[Pred]' will create an empty vector the first time
   //
-  //CachedCopyMap::iterator CCI = CopyCache.find(BBConstTy(Pred, CPV));
-  //if (CCI != CopyCache.end()) return CCI->second;
+  CachedCopyMap::iterator CCI = CopyCache.find(BBConstTy(Pred, CPV));
+  if (CCI != CopyCache.end()) return CCI->second;
   
   // Create a copy instruction and add it to the cache...
   CastInst *Inst = new CastInst(CPV, CPV->getType());
-  //CopyCache.insert(make_pair(BBConstTy(Pred, CPV), Inst));
+  CopyCache.insert(make_pair(BBConstTy(Pred, CPV), Inst));
     
   // Insert the copy just before the terminator inst of the predecessor BB
   assert(Pred->getTerminator() && "Degenerate BB encountered!");
@@ -68,13 +68,15 @@ bool HoistPHIConstants::doHoistPHIConstants(Method *M) {
       for (vector<PHINode*>::iterator PI=phis.begin(); PI != phis.end(); ++PI)
         for (unsigned i = 0; i < (*PI)->getNumIncomingValues(); ++i)
           {
-            //if (isa<ConstPoolVal>(Op)) {--- Do for all phi args -- Ruchira
+           Value *Op = (*PI)->getIncomingValue(i);
+
+            if (isa<ConstPoolVal>(Op)) {
             (*PI)->setIncomingValue(i,
                     NormalizePhiOperand((*PI),
                                         (*PI)->getIncomingValue(i),
                                         (*PI)->getIncomingBlock(i), Cache));
             Changed = true;
-            //}
+            }
           }
     }