[AMDGPU] Move RPT::getLiveRegs() check under EXPENSIVE_CHECKS
authorStanislav Mekhanoshin <Stanislav.Mekhanoshin@amd.com>
Mon, 22 Feb 2021 23:02:37 +0000 (15:02 -0800)
committerStanislav Mekhanoshin <Stanislav.Mekhanoshin@amd.com>
Mon, 22 Feb 2021 23:21:59 +0000 (15:21 -0800)
This is too expensive even for debug builds. It doubles
scheduling time if enabled.

Differential Revision: https://reviews.llvm.org/D97232

llvm/lib/Target/AMDGPU/GCNSchedStrategy.cpp

index 18a534d..0477856 100644 (file)
@@ -461,7 +461,9 @@ void GCNScheduleDAGMILive::computeBlockPressure(const MachineBasicBlock *MBB) {
     I = Rgn.first;
     auto *NonDbgMI = &*skipDebugInstructionsForward(Rgn.first, Rgn.second);
     auto LRS = BBLiveInMap.lookup(NonDbgMI);
+#ifdef EXPENSIVE_CHECKS
     assert(isEqual(getLiveRegsBefore(*NonDbgMI, *LIS), LRS));
+#endif
     RPTracker.reset(*I, &LRS);
   }