[PATCH] [GVN] Perform Scalar PRE on gep indices that feed loads before doing Load PRE
hfinkel at anl.gov
hfinkel at anl.gov
Tue Nov 4 06:39:11 PST 2014
================
Comment at: lib/Transforms/Scalar/GVN.cpp:2655
@@ -2639,3 +2654,3 @@
bool Changed = false;
-#if 0
+#if 1
// Needed for value numbering with phi construction to work.
----------------
bmakam wrote:
> hfinkel wrote:
> > If the patch turns this on, please just remove the #if.
> I will turn on the slow path and turn off the fast path in my next patch that I will upload with full context. I am not sure if we still want to keep the fast path commented out in the code or clean it up.
If you want to still keep it, add a command-line flag to enable it. We don't generally keep commented-out code, as a policy.
http://reviews.llvm.org/D6103
More information about the llvm-commits
mailing list