[PATCH] D136267: [AMDGPU] Speedup GCNDownwardRPTracker::advanceBeforeNext

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/GCNRegPressure.cpp:360
+
+  auto const RegUses = collectVirtualRegUses(*LastTrackedMI, LIS, *MRI);
+  // each RegUnit is unique in RegUses
----------------
Can you defer this until after skipDebugInstructionsForward?


================
Comment at: llvm/lib/Target/AMDGPU/GCNRegPressure.cpp:387
+      if (It != LiveRegs.end() && It->second.none())
+        LiveRegs.erase(It);
     } else if (!LI.liveAt(SI)) {
----------------
I don't understand why LiveRegs is a map in the first place


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D136267/new/

https://reviews.llvm.org/D136267