[llvm] [AMDGPU] Fix GCNUpwardRPTracker. (WIP) (PR #71186)

Valery Pykhtin via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 7 12:56:28 PST 2023


================
@@ -261,15 +257,11 @@ void GCNRPTracker::reset(const MachineInstr &MI,
   MaxPressure = CurPressure = getRegPressure(*MRI, LiveRegs);
 }
 
-void GCNUpwardRPTracker::reset(const MachineInstr &MI,
-                               const LiveRegSet *LiveRegsCopy) {
-  GCNRPTracker::reset(MI, LiveRegsCopy, true);
-}
-
 void GCNUpwardRPTracker::reset(const MachineRegisterInfo &MRI_,
                                const LiveRegSet &LiveRegs_) {
   MRI = &MRI_;
   LiveRegs = LiveRegs_;
+  LastTrackedMI = nullptr; // TODO: LastTrackedMI isnt' used, remove?
----------------
vpykhtin wrote:

I haven't noticed it's used with the base class accessor, so I decided to leave it and delete TODO comment.

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


More information about the llvm-commits mailing list