From 3e0199f7ebab06654168e72add49b7f8eed75b27 Mon Sep 17 00:00:00 2001 From: Don Hinton Date: Thu, 12 Oct 2017 16:16:06 +0000 Subject: [dump] Remove NDEBUG from test to enable dump methods [NFC] Summary: Add LLVM_FORCE_ENABLE_DUMP cmake option, and use it along with LLVM_ENABLE_ASSERTIONS to set LLVM_ENABLE_DUMP. Remove NDEBUG and only use LLVM_ENABLE_DUMP to enable dump methods. Move definition of LLVM_ENABLE_DUMP from config.h to llvm-config.h so it'll be picked up by public headers. Differential Revision: https://reviews.llvm.org/D38406 llvm-svn: 315590 --- llvm/lib/CodeGen/MachineScheduler.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'llvm/lib/CodeGen/MachineScheduler.cpp') diff --git a/llvm/lib/CodeGen/MachineScheduler.cpp b/llvm/lib/CodeGen/MachineScheduler.cpp index 674e83c..73eea42 100644 --- a/llvm/lib/CodeGen/MachineScheduler.cpp +++ b/llvm/lib/CodeGen/MachineScheduler.cpp @@ -581,7 +581,7 @@ void MachineSchedulerBase::print(raw_ostream &O, const Module* m) const { // unimplemented } -#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP) +#ifdef LLVM_ENABLE_DUMP LLVM_DUMP_METHOD void ReadyQueue::dump() const { dbgs() << "Queue " << Name << ": "; for (const SUnit *SU : Queue) @@ -919,7 +919,7 @@ void ScheduleDAGMI::placeDebugValues() { FirstDbgValue = nullptr; } -#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP) +#ifdef LLVM_ENABLE_DUMP LLVM_DUMP_METHOD void ScheduleDAGMI::dumpSchedule() const { for (MachineBasicBlock::iterator MI = begin(), ME = end(); MI != ME; ++MI) { if (SUnit *SU = getSUnit(&(*MI))) @@ -2354,7 +2354,7 @@ SUnit *SchedBoundary::pickOnlyChoice() { return nullptr; } -#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP) +#ifdef LLVM_ENABLE_DUMP // This is useful information to dump after bumpNode. // Note that the Queue contents are more useful before pickNodeFromQueue. LLVM_DUMP_METHOD void SchedBoundary::dumpScheduledState() const { @@ -2698,7 +2698,7 @@ void GenericScheduler::initPolicy(MachineBasicBlock::iterator Begin, void GenericScheduler::dumpPolicy() const { // Cannot completely remove virtual function even in release mode. -#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP) +#ifdef LLVM_ENABLE_DUMP dbgs() << "GenericScheduler RegionPolicy: " << " ShouldTrackPressure=" << RegionPolicy.ShouldTrackPressure << " OnlyTopDown=" << RegionPolicy.OnlyTopDown -- cgit v1.1