[PATCH] D22952: [LoopVectorize] Detect loops in the innermost loop before creating InnerLoopVectorizer

Nadav Rotem via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 1 16:13:41 PDT 2016


nadav accepted this revision.
This revision is now accepted and ready to land.

================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:4555
@@ -4481,3 +4554,3 @@
   // We can only vectorize innermost loops.
-  if (!TheLoop->empty()) {
-    emitAnalysis(VectorizationReport() << "loop is not the innermost loop");
+  if (hasCyclesInLoopBody()) {
+    emitAnalysis(VectorizationReport() << "the loop body has a cycle");
----------------
I think that the new message is less informative. Please make this two separate checks.  One for cycles and the other message for non-innermost. 

I don't have any other comments. 


https://reviews.llvm.org/D22952





More information about the llvm-commits mailing list