[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