Revert "[AMDGPU] Add EXPENSIVE_CHECK into GCNRPTracker::reset"
authorValery Pykhtin <valery.pykhtin@gmail.com>
Fri, 28 Oct 2022 07:15:37 +0000 (09:15 +0200)
committerValery Pykhtin <valery.pykhtin@gmail.com>
Fri, 28 Oct 2022 07:15:37 +0000 (09:15 +0200)
This reverts commit fecf067db40ffa1a6d5d665769c90cd29547f502.

The change introduces 420 test failures with EXPENSIVE_CHECK in AMDGPU which I don't want to disable.
Going to fix the failures and recommit the check.

llvm/lib/Target/AMDGPU/GCNRegPressure.cpp

index e0059e6..2f38f7f 100644 (file)
@@ -260,22 +260,6 @@ void GCNRPTracker::reset(const MachineInstr &MI,
   }
 
   MaxPressure = CurPressure = getRegPressure(*MRI, LiveRegs);
-
-  LLVM_DEBUG(dbgs() << "\nReset " << (After ? "after" : "before") << ": " << MI
-                    << print(LiveRegs, *MRI));
-#if defined(EXPENSIVE_CHECKS) && !defined(NDEBUG)
-  if (LiveRegsCopy) {
-    auto LISLRS =
-        After ? getLiveRegsAfter(MI, LIS) : getLiveRegsBefore(MI, LIS);
-    if (!isEqual(LISLRS, *LiveRegsCopy)) {
-      dbgs() << "RP mismatch: "
-             << reportMismatch(LISLRS, *LiveRegsCopy,
-                               MRI->getTargetRegisterInfo());
-      llvm_unreachable("GCNRPTracker::reset LiveRegsCopy doens't match those "
-                       "calculated by LIS");
-    }
-  }
-#endif
 }
 
 void GCNUpwardRPTracker::reset(const MachineInstr &MI,