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

[DIVYA SHANMUGHAN via Phabricator via llvm-commits]

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?


https://reviews.llvm.org/D36113