[llvm] [AMDGPU] Prefer lower total register usage in regions with spilling (PR #71882)
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 9 18:35:04 PST 2023
================
@@ -115,7 +119,19 @@ bool GCNRegPressure::less(const GCNSubtarget &ST,
SGPRImportant = false;
}
- // compare large regs pressure
+ // In regions with spilling, we should give prefernce to the schedule with
+ // less general RP.
+ if (Occ <= MFI.getMinWavesPerEU()) {
+ unsigned GPRPressure =
+ SGPRImportant ? getSGPRNum() : getVGPRNum(ST.hasGFX90AInsts());
+ unsigned OtherGPRPressure =
+ SGPRImportant ? O.getSGPRNum() : O.getVGPRNum(ST.hasGFX90AInsts());
+
+ if (GPRPressure != OtherGPRPressure)
+ return GPRPressure < OtherGPRPressure;
+ }
+
+ // Give second prefernce to less register tuple pressure
----------------
arsenm wrote:
Typo prefernce. s/less/lower/?
https://github.com/llvm/llvm-project/pull/71882
More information about the llvm-commits
mailing list