[PATCH] D32140: Global code motion of congruent computations

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/PassManagerBuilder.cpp:140
 static cl::opt<bool> EnableGVNHoist(
-    "enable-gvn-hoist", cl::init(false), cl::Hidden,
+    "enable-gvn-hoist", cl::init(true), cl::Hidden,
     cl::desc("Enable the GVN hoisting pass"));
----------------
Is it intended to be part of this patch?


================
Comment at: llvm/lib/Transforms/IPO/PassManagerBuilder.cpp:482
+  if(EnableGVNHoist)
+    MPM.add(createGVNHoistPass());
+
----------------
Can you clarify why is it inserted here? It seems like a strange place to me


https://reviews.llvm.org/D32140