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

Adam Nemet via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 1 17:27:18 PDT 2016


anemet added a subscriber: anemet.
anemet added a comment.

Where this check should be is related to PR28374.  I think it should be checked when we initially gather the inner most loops in addInnerLoop.


================
Comment at: test/Transforms/LoopVectorize/pr28541.ll:1
@@ +1,2 @@
+; RUN: opt -O2 -S < %s
+
----------------
Use -loop-vectorize rather than -O2.

================
Comment at: test/Transforms/LoopVectorize/pr28541.ll:3
@@ +2,3 @@
+
+; Check that opt does not crash on such input:
+;
----------------
mkuper wrote:
> Could you also please check that the loop doesn't get vectorized?
+1 and if you pass -pass-remarks-missed=loop-vectorize you can also check that the diag message is emitted correctly


https://reviews.llvm.org/D22952





More information about the llvm-commits mailing list