[PATCH] D98435: [LoopVectorize] Add strict in-order reduction support for fixed-width vectorization

Kerry McLaughlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 19 10:27:14 PDT 2021


kmclaughlin updated this revision to Diff 331917.
kmclaughlin marked 8 inline comments as done.
kmclaughlin edited the summary of this revision.
kmclaughlin added a comment.

- Changed the name of the flag to -enable-strict-reductions
- Removed unnecessary changes to create a mask as this was already handled by `VPReductionRecipe::execute`
- Simplified `checkOrderedReductions` for this patch. For now, if Exit is a Phi node we will not set `IsOrdered` to true.

- Removed `+sve` from the RUN line of the tests
- Added a negative test (`@fadd_multiple`)
- Added more tests


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

https://reviews.llvm.org/D98435

Files:
  llvm/include/llvm/Analysis/IVDescriptors.h
  llvm/include/llvm/Transforms/Utils/LoopUtils.h
  llvm/lib/Analysis/IVDescriptors.cpp
  llvm/lib/Transforms/Utils/LoopUtils.cpp
  llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
  llvm/test/Transforms/LoopVectorize/AArch64/strict-fadd.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D98435.331917.patch
Type: text/x-patch
Size: 24336 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210319/b6dbd046/attachment.bin>


More information about the llvm-commits mailing list