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

Kerry McLaughlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 21 08:12:41 PST 2022


kmclaughlin updated this revision to Diff 410309.
kmclaughlin marked 4 inline comments as done.
kmclaughlin added a comment.

- Changes to tryToBlend() so that all incoming values of the Phi are checked for in-loop reductions.
- Added an assert to tryToBlend() that the number of incoming values to the Phi is 2 if an in-loop reduction is found. Also added an assert that only one of the incoming values is an in-loop reduction.
- Reworded the comment describing the number of uses of Phi in getReductionOpChain().


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.410309.patch
Type: text/x-patch
Size: 81534 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220221/0af6c47d/attachment.bin>


More information about the llvm-commits mailing list