[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