[PATCH] D42447: [LV][VPlan] Detect outer loops for explicit vectorization.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 18 10:39:29 PDT 2018


fhahn accepted this revision.
fhahn added a comment.
This revision is now accepted and ready to land.

Thanks Diego and thanks for your patience! LGTM, but please wait a bit with committing, in case other people people want to raise any additional comments.



================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:8629
+
+  if (!EnableVPlanNativePath) {
+    DEBUG(dbgs() << "LV: Not vectorizing. VPlan-native path is disabled.");
----------------
I suppose we should never call processLoopInVPlanNativePath without the flag? Could this be an assertion?


https://reviews.llvm.org/D42447





More information about the llvm-commits mailing list