[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 llvm-commits at lists.llvm.org
Sun Aug 14 12:57:32 PDT 2022


mingmingl added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp:1981
+                                                   Type *Val, unsigned Index) {
+  InstructionCost cost = this->getVectorInstrCost(I.getOpcode(), Val, Index);
+
----------------
RKSimon wrote:
> do you need the this-> ?
Ah `this->` is not needed, fixed it.

(Sorry for the delay; I was out in the past week)


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

https://reviews.llvm.org/D128302



More information about the llvm-commits mailing list