[llvm] [AMDGPU] Use correct VGPR threshold for flagging ExcessRP regions in unified register file case (PR #85860)
Stanislav Mekhanoshin via llvm-commits
llvm-commits at lists.llvm.org
Fri Mar 22 10:24:46 PDT 2024
================
@@ -974,11 +974,13 @@ void GCNSchedStage::checkScheduling() {
<< DAG.MinOccupancy << ".\n");
}
- unsigned MaxVGPRs = ST.getMaxNumVGPRs(MF);
+ unsigned MaxVGPRs = ST.getMaxNumVGPRs(MF, /*WholeRegisterFile*/ true);
+ unsigned MaxArchVGPRs = ST.getMaxNumVGPRs(MF, /*WholeRegisterFile*/ false);
----------------
rampitec wrote:
The new interface is quite confusing and limited for just these few targets. Then I do not believe we have a lot of use for PSets since using our own RP tracker. If a small contained change can do the same thing, I would prefer a small contained change.
https://github.com/llvm/llvm-project/pull/85860
More information about the llvm-commits
mailing list