[PATCH] D95245: [SVE] Add support for scalable vectorization of loops with int/fast FP reductions

[Kerry McLaughlin via Phabricator via llvm-commits]

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

- Removed changes to `LoopVectorizationPlanner::plan` and instead check whether reductions can be vectorized in `computeFeasibleMaxVF`. If any reduction in the loop cannot be vectorized with a scalable VF, we fall back on fixed-width vectorization.

- Changes to have `VectorizationFactor` use `InstructionCost` were not necessary to the patch after the above change and have also been removed.

- Improved the tests in `scalable_reductions.ll` based on suggestions from @fhahn & @david-arm


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

https://reviews.llvm.org/D95245

Files:
  llvm/include/llvm/Analysis/TargetTransformInfo.h
  llvm/include/llvm/Analysis/TargetTransformInfoImpl.h
  llvm/lib/Analysis/TargetTransformInfo.cpp
  llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp
  llvm/lib/Target/AArch64/AArch64TargetTransformInfo.h
  llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
  llvm/test/Transforms/LoopVectorize/AArch64/scalable_reductions.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D95245.319303.patch
Type: text/x-patch
Size: 27491 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210126/1660934b/attachment.bin>