From 1eb4ec6a2eedec4e69b664540fa1a14d5bf899b4 Mon Sep 17 00:00:00 2001 From: Xinliang David Li Date: Thu, 2 Feb 2017 19:18:56 +0000 Subject: [PGO] make graph view internal options available for all builds Differential Revision: https://reviews.llvm.org/D29259 llvm-svn: 293921 --- llvm/lib/Analysis/BlockFrequencyInfo.cpp | 10 ---------- 1 file changed, 10 deletions(-) (limited to 'llvm/lib/Analysis/BlockFrequencyInfo.cpp') diff --git a/llvm/lib/Analysis/BlockFrequencyInfo.cpp b/llvm/lib/Analysis/BlockFrequencyInfo.cpp index 3c57c171e07..2064be577f6 100644 --- a/llvm/lib/Analysis/BlockFrequencyInfo.cpp +++ b/llvm/lib/Analysis/BlockFrequencyInfo.cpp @@ -26,7 +26,6 @@ using namespace llvm; #define DEBUG_TYPE "block-freq" -#ifndef NDEBUG static cl::opt ViewBlockFreqPropagationDAG( "view-block-freq-propagation-dags", cl::Hidden, cl::desc("Pop up a window to show a dag displaying how block " @@ -116,7 +115,6 @@ struct DOTGraphTraits : public BFIDOTGTraitsBase { }; } // end namespace llvm -#endif BlockFrequencyInfo::BlockFrequencyInfo() {} @@ -156,13 +154,11 @@ void BlockFrequencyInfo::calculate(const Function &F, if (!BFI) BFI.reset(new ImplType); BFI->calculate(F, BPI, LI); -#ifndef NDEBUG if (ViewBlockFreqPropagationDAG != GVDT_None && (ViewBlockFreqFuncName.empty() || F.getName().equals(ViewBlockFreqFuncName))) { view(); } -#endif } BlockFrequency BlockFrequencyInfo::getBlockFreq(const BasicBlock *BB) const { @@ -214,13 +210,7 @@ void BlockFrequencyInfo::setBlockFreqAndScale( /// Pop up a ghostview window with the current block frequency propagation /// rendered using dot. void BlockFrequencyInfo::view() const { -// This code is only for debugging. -#ifndef NDEBUG ViewGraph(const_cast(this), "BlockFrequencyDAGs"); -#else - errs() << "BlockFrequencyInfo::view is only available in debug builds on " - "systems with Graphviz or gv!\n"; -#endif // NDEBUG } const Function *BlockFrequencyInfo::getFunction() const { -- cgit v1.2.3