[PATCH] D105632: [LV] Use lookThroughAnd with logical reductions

Kerry McLaughlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 14 09:57:26 PDT 2021


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

- Changed `AddReductionVar` so that for min & max reductions where `lookThroughAnd` has set the Start instruction to the user of the Phi, we do not change the return type. This ensures that we create the reduction with the wider type (e.g. i32) and truncate the result back to the smaller type (e.g. i8).


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

https://reviews.llvm.org/D105632

Files:
  llvm/lib/Analysis/IVDescriptors.cpp
  llvm/test/Transforms/LoopVectorize/reduction-inloop-pred.ll
  llvm/test/Transforms/LoopVectorize/reduction-inloop.ll
  llvm/test/Transforms/LoopVectorize/trunc-reductions.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D105632.358651.patch
Type: text/x-patch
Size: 30928 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210714/34e55275/attachment.bin>


More information about the llvm-commits mailing list