[PATCH] D128302: [AArch64][CostModel] Detects that ExtractElement at index is not free in AArch64 when result is used as integer.

[Mingming Liu via Phabricator via llvm-commits]

mingmingl updated this revision to Diff 455486.
mingmingl marked 3 inline comments as done.
mingmingl edited the summary of this revision.
mingmingl added a comment.

Add a private method `AArch64TTIImpl::getVectorInstrCostHelper` as a helper function. This way, instruction-based method could share the core logic (e.g.,
returning zero cost if type is legalized to scalar).

Uploaded with full context and updated all affected test cases.


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

https://reviews.llvm.org/D128302

Files:
  llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp
  llvm/lib/Target/AArch64/AArch64TargetTransformInfo.h
  llvm/test/Analysis/CostModel/AArch64/kryo-inseltpoison.ll
  llvm/test/Analysis/CostModel/AArch64/kryo.ll
  llvm/test/Analysis/CostModel/AArch64/sve-insert-extract.ll
  llvm/test/Transforms/LICM/AArch64/extract-element.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D128302.455486.patch
Type: text/x-patch
Size: 16639 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220825/a3437805/attachment.bin>