[llvm] [NFC][AMDGPU] Move `advanceBeforeNext` before `advanceToNext` (PR #182808)

Shilei Tian via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 22 22:03:56 PST 2026


https://github.com/shiltian updated https://github.com/llvm/llvm-project/pull/182808

>From dd271382450e51f2184da6f7484fe8b787ac77bc Mon Sep 17 00:00:00 2001
From: Shilei Tian <i at tianshilei.me>
Date: Mon, 23 Feb 2026 01:01:23 -0500
Subject: [PATCH] [NFC][AMDGPU] Move `advanceBeforeNext` before `advanceToNext`

The document of `advanceToNext` says, `advanceBeforeNext` has to be called first.
---
 llvm/lib/Target/AMDGPU/GCNSchedStrategy.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/llvm/lib/Target/AMDGPU/GCNSchedStrategy.cpp b/llvm/lib/Target/AMDGPU/GCNSchedStrategy.cpp
index 277ac7d4ecf4c..f3c471bb697d2 100644
--- a/llvm/lib/Target/AMDGPU/GCNSchedStrategy.cpp
+++ b/llvm/lib/Target/AMDGPU/GCNSchedStrategy.cpp
@@ -1061,16 +1061,16 @@ void GCNScheduleDAGMILive::computeBlockPressure(unsigned RegionIdx,
       auto &Rgn = Regions[CurRegion];
       NonDbgMI = &*skipDebugInstructionsForward(Rgn.first, Rgn.second);
     }
-    RPTracker.advanceToNext();
     RPTracker.advanceBeforeNext();
+    RPTracker.advanceToNext();
   }
 
   if (OnlySucc) {
     if (I != MBB->end()) {
+      RPTracker.advanceBeforeNext();
       RPTracker.advanceToNext();
       RPTracker.advance(MBB->end());
     }
-    RPTracker.advanceBeforeNext();
     MBBLiveIns[OnlySucc] = RPTracker.moveLiveRegs();
   }
 }



More information about the llvm-commits mailing list