[PATCH] D129072: [RISCV] Fix the scale of getIntMatCost and also need adjust for getIntImmCost
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 4 19:07:12 PDT 2022
craig.topper added inline comments.
================
Comment at: llvm/lib/Target/RISCV/MCTargetDesc/RISCVMatInt.cpp:395
}
- return std::max(1, Cost);
+ return std::max(100, Cost);
}
----------------
Doesn’t this prevent the function from being able to return 75 like it should for a single compressed instruction?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D129072/new/
https://reviews.llvm.org/D129072
More information about the llvm-commits
mailing list