[llvm-commits] [llvm] r87042 - /llvm/trunk/lib/Transforms/Scalar/SCCVN.cpp

[Owen Anderson]

Author: resistor
Date: Thu Nov 12 17:22:41 2009
New Revision: 87042

URL: http://llvm.org/viewvc/llvm-project?rev=87042&view=rev
Log:
Re-enable this code, since redundant PHIs are now being better nuked.

Modified:
    llvm/trunk/lib/Transforms/Scalar/SCCVN.cpp

Modified: llvm/trunk/lib/Transforms/Scalar/SCCVN.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/SCCVN.cpp?rev=87042&r1=87041&r2=87042&view=diff

==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/SCCVN.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/SCCVN.cpp Thu Nov 12 17:22:41 2009
@@ -629,9 +629,6 @@
     }
   }
 
-  // FIXME: This code is commented out for now, because it can lead to the
-  // insertion of a lot of redundant PHIs being inserted by SSAUpdater.
-#if 0
   // Perform a forward data-flow to compute availability at all points on
   // the CFG.
   do {
@@ -709,7 +706,6 @@
       CurInst->eraseFromParent();
     }
   }
-#endif
 
   VT.clear();
   for (DenseMap<BasicBlock*, ValueNumberScope*>::iterator