[PATCH] D117884: [RISCV] Add cost modelling for masked memory op

Alex via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 24 22:21:31 PST 2022


alextsao1999 added a reviewer: kito-cheng.
alextsao1999 added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp:173
+
+  std::pair<InstructionCost, MVT> LT = TLI->getTypeLegalizationCost(DL, Src);
+  return LT.first;
----------------
kito-cheng wrote:
> That seems like not included the memory access cost? how about `return getMemoryOpCost (Opcode, Src, Alignment, AddressSpace, CostKind)` here?
Thanks, I fixed it


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117884



More information about the llvm-commits mailing list