[llvm] [RISCV][RegAlloc] Add getCSRFirstUseCost for RISC-V (PR #131349)
Michael Maitland via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 20 08:25:40 PDT 2025
================
@@ -2375,10 +2375,12 @@ void RAGreedy::aboutToRemoveInterval(const LiveInterval &LI) {
}
void RAGreedy::initializeCSRCost() {
- // We use the larger one out of the command-line option and the value report
- // by TRI.
+ // We use the command-line option if it is explicitly set, otherwise use the
+ // larger one out of the command-line option and the value reported by TRI.
CSRCost = BlockFrequency(
- std::max((unsigned)CSRFirstTimeCost, TRI->getCSRFirstUseCost()));
+ CSRFirstTimeCost.getNumOccurrences()
+ ? CSRFirstTimeCost
+ : std::max((unsigned)CSRFirstTimeCost, TRI->getCSRFirstUseCost()));
----------------
michaelmaitland wrote:
If someone goes in and changes the default value, wouldn't we want the comparison to occur without having to change the code here?
https://github.com/llvm/llvm-project/pull/131349
More information about the llvm-commits
mailing list