[PATCH] D91718: [LV] Legalize scalable VF hints

Cullen Rhodes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 10 13:01:33 PST 2020


c-rhodes updated this revision to Diff 310996.
c-rhodes edited the summary of this revision.
c-rhodes added a comment.

Changes:

- Rebase since D88962 <https://reviews.llvm.org/D88962> and D91077 <https://reviews.llvm.org/D91077> have now landed.
- Address @sdesmalen's comments. Added a target hook `supportsScalableVectors` in D93060 <https://reviews.llvm.org/D93060> which is used in this patch to bail out of vectorization if unsupported. Also changed the behavior when scalable vectorization is unfeasible given a dependency, it now changes the hint to fixed, where existing behavior applies (i.e. clamping).
- Updated tests.


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

https://reviews.llvm.org/D91718

Files:
  llvm/include/llvm/Analysis/TargetTransformInfo.h
  llvm/include/llvm/Analysis/TargetTransformInfoImpl.h
  llvm/include/llvm/CodeGen/BasicTTIImpl.h
  llvm/lib/Analysis/TargetTransformInfo.cpp
  llvm/lib/Target/AArch64/AArch64TargetTransformInfo.h
  llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
  llvm/test/Transforms/LoopVectorize/AArch64/scalable-vf-hint.ll
  llvm/test/Transforms/LoopVectorize/scalable-vf-hint.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D91718.310996.patch
Type: text/x-patch
Size: 21936 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201210/e9145c26/attachment.bin>


More information about the llvm-commits mailing list