[PATCH] D79783: [LoopVectorize] Fallback to a scalar epilogue when TP fails

[Pierre van Houtryve via Phabricator via llvm-commits]

Pierre-vh updated this revision to Diff 264822.
Pierre-vh marked 3 inline comments as done.
Pierre-vh added a comment.

- Changing implementation of the patch following discussion
- Removed the `ReportFailure` argument of `prepareToFoldTailByMasking`. I don't think it's useful anymore, but feedback is welcome. (The only thing that annoys me is that we now print "loop not vectorized" even when we'll fallback to a scalar epilogue)
- Added a test that makes use of the attribute that enables tail-folding
- Simplified tests


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

https://reviews.llvm.org/D79783

Files:
  llvm/include/llvm/Transforms/Vectorize/LoopVectorizationLegality.h
  llvm/lib/Transforms/Vectorize/LoopVectorizationLegality.cpp
  llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
  llvm/test/Transforms/LoopVectorize/ARM/tail-folding-scalar-epilogue-fallback.ll
  llvm/test/Transforms/LoopVectorize/use-scalar-epilogue-if-tp-fails.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D79783.264822.patch
Type: text/x-patch
Size: 18847 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200519/b9d154e4/attachment.bin>