[llvm] r290644 - [PM] Disable the loop vectorizer from the new PM's pipeline as it

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 27 18:24:55 PST 2016


Author: chandlerc
Date: Tue Dec 27 20:24:55 2016
New Revision: 290644

URL: http://llvm.org/viewvc/llvm-project?rev=290644&view=rev
Log:
[PM] Disable the loop vectorizer from the new PM's pipeline as it
currenty relies on the old PM's dependency system forming LCSSA.

The new PM will require a different design for this, and for now this is
causing most of the issues I'm currently seeing in testing. I'd like to
get to a testable baseline and then work on re-enabling things one at
a time.

Modified:
    llvm/trunk/lib/Passes/PassBuilder.cpp

Modified: llvm/trunk/lib/Passes/PassBuilder.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Passes/PassBuilder.cpp?rev=290644&r1=290643&r2=290644&view=diff
==============================================================================
--- llvm/trunk/lib/Passes/PassBuilder.cpp (original)
+++ llvm/trunk/lib/Passes/PassBuilder.cpp Tue Dec 27 20:24:55 2016
@@ -490,7 +490,11 @@ PassBuilder::buildPerModuleDefaultPipeli
   // rather than on each loop in an inside-out manner, and so they are actually
   // function passes.
   OptimizePM.addPass(LoopDistributePass());
+#if 0
+  // FIXME: LoopVectorize relies on "requiring" LCSSA which isn't supported in
+  // the new PM.
   OptimizePM.addPass(LoopVectorizePass());
+#endif
   // FIXME: Need to port Loop Load Elimination and add it here.
   OptimizePM.addPass(InstCombinePass());
 




More information about the llvm-commits mailing list