From bb04f6e28fe49046975297846548dc99e68c82b9 Mon Sep 17 00:00:00 2001 From: Sanjoy Das Date: Thu, 28 Jan 2016 23:49:27 +0000 Subject: [PATCH] [PlaceSafepoints] Use DEBUG() instead of TraceLSP DEBUG() is the more idiomatic LLVM style. llvm-svn: 259121 --- llvm/lib/Transforms/Scalar/PlaceSafepoints.cpp | 14 +++----------- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/llvm/lib/Transforms/Scalar/PlaceSafepoints.cpp b/llvm/lib/Transforms/Scalar/PlaceSafepoints.cpp index 7b3b633..1cadab6 100644 --- a/llvm/lib/Transforms/Scalar/PlaceSafepoints.cpp +++ b/llvm/lib/Transforms/Scalar/PlaceSafepoints.cpp @@ -95,9 +95,6 @@ static cl::opt CountedLoopTripWidth("spp-counted-loop-trip-width", static cl::opt SplitBackedge("spp-split-backedge", cl::Hidden, cl::init(false)); -// Print tracing output -static cl::opt TraceLSP("spp-trace", cl::Hidden, cl::init(false)); - namespace { /// An analysis pass whose purpose is to identify each of the backedges in @@ -320,8 +317,7 @@ bool PlaceBackedgeSafepointsImpl::runOnLoop(Loop *L) { // avoiding the runtime cost of the actual safepoint. if (!AllBackedges) { if (mustBeFiniteCountedLoop(L, SE, Pred)) { - if (TraceLSP) - errs() << "skipping safepoint placement in finite loop\n"; + DEBUG(dbgs() << "skipping safepoint placement in finite loop\n"); FiniteExecution++; continue; } @@ -330,8 +326,7 @@ bool PlaceBackedgeSafepointsImpl::runOnLoop(Loop *L) { // Note: This is only semantically legal since we won't do any further // IPO or inlining before the actual call insertion.. If we hadn't, we // might latter loose this call safepoint. - if (TraceLSP) - errs() << "skipping safepoint placement due to unconditional call\n"; + DEBUG(dbgs() << "skipping safepoint placement due to unconditional call\n"); CallInLoop++; continue; } @@ -347,10 +342,7 @@ bool PlaceBackedgeSafepointsImpl::runOnLoop(Loop *L) { // variables) and branches to the true header TerminatorInst *Term = Pred->getTerminator(); - if (TraceLSP) { - errs() << "[LSP] terminator instruction: "; - Term->dump(); - } + DEBUG(dbgs() << "[LSP] terminator instruction: " << *Term); PollLocations.push_back(Term); } -- 2.7.4