[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