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

[Cullen Rhodes via Phabricator via llvm-commits]

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

Changes:

- `s/FixedVF/MaxSafeElements/`
- Rather than bail out, clamp to fixed VF is scalable is unfeasible. Remove change to return `Optional<ElementCount>` from `LoopVectorizationCostModel::computeFeasibleMaxVF`.
- In `LoopVectorizationPlanner::plan`, if a `UserVF` is specified but isn't legal then use `MaxVF` (earlier).
- Simplified the IR in the 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.308062.patch
Type: text/x-patch
Size: 17454 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201127/24fdf22c/attachment.bin>