From 726e12cf0ce6c926877bd8e19e6afe05e5101eeb Mon Sep 17 00:00:00 2001 From: Matthias Braun Date: Wed, 19 Sep 2018 00:23:35 +0000 Subject: ScheduleDAG: Cleanup dumping code; NFC - Instead of having both `SUnit::dump(ScheduleDAG*)` and `ScheduleDAG::dumpNode(ScheduleDAG*)`, just keep the latter around. - Add `ScheduleDAG::dump()` and avoid code duplication in several places. Implement it for different ScheduleDAG variants. - Add `ScheduleDAG::dumpNodeName()` in favor of the `SUnit::print()` functions. They were only ever used for debug dumping and putting the function into ScheduleDAG is consistent with the `dumpNode()` change. llvm-svn: 342520 --- llvm/lib/Target/AMDGPU/GCNMinRegStrategy.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'llvm/lib/Target/AMDGPU/GCNMinRegStrategy.cpp') diff --git a/llvm/lib/Target/AMDGPU/GCNMinRegStrategy.cpp b/llvm/lib/Target/AMDGPU/GCNMinRegStrategy.cpp index 192d534bb9c..ec6bcae3355 100644 --- a/llvm/lib/Target/AMDGPU/GCNMinRegStrategy.cpp +++ b/llvm/lib/Target/AMDGPU/GCNMinRegStrategy.cpp @@ -258,7 +258,7 @@ GCNMinRegScheduler::schedule(ArrayRef TopRoots, assert(C); RQ.remove(*C); auto SU = C->SU; - LLVM_DEBUG(dbgs() << "Selected "; SU->dump(&DAG)); + LLVM_DEBUG(dbgs() << "Selected "; DAG.dumpNode(*SU)); releaseSuccessors(SU, StepNo); Schedule.push_back(SU); -- cgit v1.2.3