[PATCH] D117580: [LoopVectorize] Support conditional in-loop vector reductions

Kerry McLaughlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 3 10:03:00 PST 2022


kmclaughlin updated this revision to Diff 405685.
kmclaughlin added a comment.

- Changed getNextInstruction to iterate over `Cur->users()` and handle Phi nodes found by moving to the next user, similar to ICmp/FCmp.
- Removed the `dyn_cast<PHINode>(Cur) == LoopExitInstr` block as Phis are now handled by getNextInstruction.
- Added tests for various scenarios involving chained reductions where we should not vectorise with in-loop reductions.


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

https://reviews.llvm.org/D117580

Files:
  llvm/lib/Analysis/IVDescriptors.cpp
  llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
  llvm/test/Transforms/LoopVectorize/AArch64/scalable-reduction-inloop-cond.ll
  llvm/test/Transforms/LoopVectorize/AArch64/sve-tail-folding.ll
  llvm/test/Transforms/LoopVectorize/reduction-inloop-cond.ll
  llvm/test/Transforms/LoopVectorize/reduction-inloop-uf4.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D117580.405685.patch
Type: text/x-patch
Size: 119091 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220203/8ae36bca/attachment.bin>


More information about the llvm-commits mailing list