[PATCH] D102515: [CostModel] Return an invalid cost for memory ops with unsupported types

Kerry McLaughlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 2 07:36:56 PDT 2021


kmclaughlin updated this revision to Diff 349264.
kmclaughlin edited the summary of this revision.
kmclaughlin added a comment.

- Changed getTypeConversion to only set the cost to Invalid for `TypeScalarizeScalableVector`
- Removed a failing test in AArch64SelectionDAGTest.cpp testing the `"Cannot legalize this vector"` assert, which has been removed.
- Removed the `CHECK-NO-MAX-VSCALE` RUN line from scalable-vf-hint.ll. This line used the `-force-target-supports-scalable-vectors=true` flag but did not enable `+sve`, resulting in an Invalid cost being returning in `getRegUsage`.


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

https://reviews.llvm.org/D102515

Files:
  llvm/lib/CodeGen/TargetLoweringBase.cpp
  llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp
  llvm/test/Analysis/CostModel/AArch64/sve-illegal-types.ll
  llvm/test/Transforms/LoopVectorize/AArch64/scalable-vf-hint.ll
  llvm/unittests/CodeGen/AArch64SelectionDAGTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D102515.349264.patch
Type: text/x-patch
Size: 10210 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210602/86e65fd3/attachment.bin>


More information about the llvm-commits mailing list