[PATCH] D36113: [Loop Vectorize] Vectorize Loops with Backward Dependence

DIVYA SHANMUGHAN via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 10 15:17:48 PDT 2017

DIVYA marked 2 inline comments as done.
DIVYA added inline comments.

Comment at: include/llvm/Transforms/IPO/PassManagerBuilder.h:151
   bool LoopVectorize;
+  bool LoopVectorizePred;
   bool RerollLoops;
hfinkel wrote:
> I don't think we need a special pass-manager option for this. We can put it behind a cl::opt flag, but once it is judged to be ready for the default pipeline, I imagine we'll always enable it whenever vectorization is enabled (at least at -O3).
I had already added cl::opt flag "EnableLoopVectorizePred" in PassManagerBuilder.cpp to have an option to disable the pass.Do I need add to anything else?


More information about the llvm-commits mailing list