[PATCH] D63433: [RISCV] Add RISCV-specific TargetTransformInfo

[Juergen Ributzka via Phabricator via llvm-commits]

ributzka added a comment.

If I recall correctly, then getIntImmCost(const APInt &Imm) predates the ConstantHoisting pass and is also used by other optimizations. The purpose of that hook is to calculate the materialization cost of a constant and is context free.

I added getIntImmCost(unsigned Opcode/Intrinsic::ID IID, unsigned Idx, const APInt &Imm, Type *Ty), because many instructions on AArch64 were able to fold constants and getIntImmCost(const APInt &Imm) would not provide accurate information to identify expensive constants.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63433