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

[David Li via Phabricator via llvm-commits]

davidxl added a comment.

I want to suggest a different way of changing the interfaces to make the intention clearer:

1. add a new interface for vectorcost in TTI that takes an instruction (to replace the opcode parameter)
2. for places where context instruction is not available, no changes are needed;
3. in TTIImpl, by default, the instruction based interface is simply a wrapper to the opcode based interface
4. for aarch64, additional logic is added for checking the lane ...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128302