[llvm] [AMDGPU] Optionally Use GCNRPTrackers during scheduling (PR #93090)

Valery Pykhtin via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 4 02:04:46 PDT 2024


================
@@ -357,6 +393,16 @@ SUnit *GCNSchedStrategy::pickNode(bool &IsTopNode) {
   return SU;
 }
 
+void GCNSchedStrategy::schedNode(SUnit *SU, bool IsTopNode) {
+  if (GCNTrackers) {
+    MachineInstr *MI = SU->getInstr();
+    IsTopNode ? (void)DownwardTracker.advance(MI, false, DAG->getLIS())
+              : UpwardTracker.recede(*MI, false);
----------------
vpykhtin wrote:

To me, if the node is scheduled then it should be "tracked"? If so, then the flag ShouldTrackIt isn't needed because the only use left is temp trackers where we shouldn't care?

https://github.com/llvm/llvm-project/pull/93090


More information about the llvm-commits mailing list