[PATCH] D93317: [LV] Vectorize (some) early and multiple exit loops

[Philip Reames via Phabricator via llvm-commits]

reames updated this revision to Diff 311986.
reames added a comment.

Update tests to cover cases where we can't vectorize due to either a) size, or b) predication.

Doing this revealed that the handling of the predicate don't vectorize option is broken in the patch.  (We correctly vectorize with a scalar epilogue where the user intent was not to vectorize.)  However, staring at it a bit, the problem seems to exist in the existing code as well.  I'm going to treat that as an entirely separate issue unless reviewers insist.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93317/new/

https://reviews.llvm.org/D93317

Files:
  llvm/lib/Transforms/Vectorize/LoopVectorizationLegality.cpp
  llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
  llvm/test/Transforms/LoopVectorize/control-flow.ll
  llvm/test/Transforms/LoopVectorize/loop-form.ll
  llvm/test/Transforms/LoopVectorize/loop-legality-checks.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D93317.311986.patch
Type: text/x-patch
Size: 34415 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201215/97a75ddc/attachment.bin>