[llvm] [AMDGPU] Optionally Use GCNRPTrackers during scheduling (PR #93090)
Valery Pykhtin via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 4 01:52:09 PDT 2024
================
@@ -213,8 +242,15 @@ void GCNSchedStrategy::pickNodeFromQueue(SchedBoundary &Zone,
unsigned SGPRPressure = 0;
unsigned VGPRPressure = 0;
if (DAG->isTrackingPressure()) {
- SGPRPressure = Pressure[AMDGPU::RegisterPressureSets::SReg_32];
- VGPRPressure = Pressure[AMDGPU::RegisterPressureSets::VGPR_32];
+ SGPRPressure =
----------------
vpykhtin wrote:
This is hard to read. Can you do something like:
```
if (!GCNTrackers) {
// old code
} else {
GCNRPTracker *T = isTop? &DownwardTracker : &UpwardTracker;
... = T->getPressure().getVGPRNum(false);
... = T->getPressure().getSGPRNum();
}
```
https://github.com/llvm/llvm-project/pull/93090
More information about the llvm-commits
mailing list