[PATCH] D114950: [RISCV] Promote large integers to constant pool

[Zixuan Wu via Phabricator via llvm-commits]

zixuan-wu added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVSubtarget.cpp:113
 
+unsigned RISCVSubtarget::getMaxBuildIntCost() const {
+  // Loading integer from constant pool needs two instructions: an address
----------------
Since we can get schedmodel and load latency, is there still need that such API to adapt different subtarget cost? Other body can give some suggestion to talk.  


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114950