From fb69810a2b617d888b6ea7c6a69fee7364fe233b Mon Sep 17 00:00:00 2001 From: Jakob Stoklund Olesen Date: Fri, 4 Mar 2011 18:08:26 +0000 Subject: [PATCH] Tweak debug output. No functional changes. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@127006 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/CodeGen/RegAllocGreedy.cpp | 7 ++++++- lib/CodeGen/SplitKit.cpp | 9 --------- 2 files changed, 6 insertions(+), 10 deletions(-) diff --git a/lib/CodeGen/RegAllocGreedy.cpp b/lib/CodeGen/RegAllocGreedy.cpp index eada8f9910b..14e376db605 100644 --- a/lib/CodeGen/RegAllocGreedy.cpp +++ b/lib/CodeGen/RegAllocGreedy.cpp @@ -627,7 +627,12 @@ float RAGreedy::calcGlobalSplitCost(const BitVector &LiveBundles) { if (Inserts) GlobalCost += Inserts * SpillPlacer->getBlockFrequency(BC.Number); } - DEBUG(dbgs() << "Global cost = " << GlobalCost << '\n'); + DEBUG({ + dbgs() << "Global cost = " << GlobalCost << " with bundles"; + for (int i = LiveBundles.find_first(); i>=0; i = LiveBundles.find_next(i)) + dbgs() << " EB#" << i; + dbgs() << ".\n"; + }); return GlobalCost; } diff --git a/lib/CodeGen/SplitKit.cpp b/lib/CodeGen/SplitKit.cpp index f16d0338b66..4adf8f7f949 100644 --- a/lib/CodeGen/SplitKit.cpp +++ b/lib/CodeGen/SplitKit.cpp @@ -314,8 +314,6 @@ void SplitEditor::extendRange(unsigned RegIdx, SlotIndex Idx) { // and we may need to create even more phi-defs to preserve VNInfo SSA form. // Perform a search for all predecessor blocks where we know the dominating // VNInfo. Insert phi-def VNInfos along the path back to IdxMBB. - DEBUG(dbgs() << "\n Reaching defs for BB#" << IdxMBB->getNumber() - << " at " << Idx << " in " << *LI << '\n'); // Initialize the live-out cache the first time it is needed. if (LiveOutSeen.empty()) { @@ -413,7 +411,6 @@ VNInfo *SplitEditor::updateSSA(unsigned RegIdx, unsigned Changes; do { Changes = 0; - DEBUG(dbgs() << " Iterating over " << LiveIn.size() << " blocks.\n"); // Propagate live-out values down the dominator tree, inserting phi-defs // when necessary. Since LiveIn was created by a BFS, going backwards makes // it more likely for us to visit immediate dominators before their @@ -454,8 +451,6 @@ VNInfo *SplitEditor::updateSSA(unsigned RegIdx, SlotIndex Start = LIS.getMBBStartIdx(MBB); VNInfo *VNI = LI->getNextValue(Start, 0, LIS.getVNInfoAllocator()); VNI->setIsPHIDef(true); - DEBUG(dbgs() << " - BB#" << MBB->getNumber() - << " phi-def #" << VNI->id << " at " << Start << '\n'); // We no longer need LI to be live-in. LiveIn.erase(LiveIn.begin()+(i-1)); // Blocks in LiveIn are either IdxMBB, or have a value live-through. @@ -487,13 +482,9 @@ VNInfo *SplitEditor::updateSSA(unsigned RegIdx, if (LOP.second != Node && LOP.first != IDomValue.first) { ++Changes; LOP = IDomValue; - DEBUG(dbgs() << " - BB#" << MBB->getNumber() - << " idom valno #" << IDomValue.first->id - << " from BB#" << IDom->getBlock()->getNumber() << '\n'); } } } - DEBUG(dbgs() << " - made " << Changes << " changes.\n"); } while (Changes); assert(IdxVNI && "Didn't find value for Idx");