[llvm] r293921 - [PGO] make graph view internal options available for all builds
Xinliang David Li via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 2 11:18:57 PST 2017
Author: davidxl
Date: Thu Feb 2 13:18:56 2017
New Revision: 293921
URL: http://llvm.org/viewvc/llvm-project?rev=293921&view=rev
Log:
[PGO] make graph view internal options available for all builds
Differential Revision: https://reviews.llvm.org/D29259
Modified:
llvm/trunk/lib/Analysis/BlockFrequencyInfo.cpp
llvm/trunk/lib/CodeGen/MachineBlockFrequencyInfo.cpp
llvm/trunk/lib/CodeGen/MachineBlockPlacement.cpp
llvm/trunk/lib/Transforms/Instrumentation/PGOInstrumentation.cpp
Modified: llvm/trunk/lib/Analysis/BlockFrequencyInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/BlockFrequencyInfo.cpp?rev=293921&r1=293920&r2=293921&view=diff
==============================================================================
--- llvm/trunk/lib/Analysis/BlockFrequencyInfo.cpp (original)
+++ llvm/trunk/lib/Analysis/BlockFrequencyInfo.cpp Thu Feb 2 13:18:56 2017
@@ -26,7 +26,6 @@ using namespace llvm;
#define DEBUG_TYPE "block-freq"
-#ifndef NDEBUG
static cl::opt<GVDAGType> 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<BlockFrequencyInfo
};
} // end namespace llvm
-#endif
BlockFrequencyInfo::BlockFrequencyInfo() {}
@@ -156,13 +154,11 @@ void BlockFrequencyInfo::calculate(const
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::setBlockFreqAnd
/// 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<BlockFrequencyInfo *>(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 {
Modified: llvm/trunk/lib/CodeGen/MachineBlockFrequencyInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/MachineBlockFrequencyInfo.cpp?rev=293921&r1=293920&r2=293921&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/MachineBlockFrequencyInfo.cpp (original)
+++ llvm/trunk/lib/CodeGen/MachineBlockFrequencyInfo.cpp Thu Feb 2 13:18:56 2017
@@ -28,7 +28,6 @@ using namespace llvm;
#define DEBUG_TYPE "block-freq"
-#ifndef NDEBUG
static cl::opt<GVDAGType> ViewMachineBlockFreqPropagationDAG(
"view-machine-block-freq-propagation-dags", cl::Hidden,
@@ -145,7 +144,6 @@ struct DOTGraphTraits<MachineBlockFreque
};
} // end namespace llvm
-#endif
INITIALIZE_PASS_BEGIN(MachineBlockFrequencyInfo, "machine-block-freq",
"Machine Block Frequency Analysis", true, true)
@@ -177,13 +175,11 @@ bool MachineBlockFrequencyInfo::runOnMac
if (!MBFI)
MBFI.reset(new ImplType);
MBFI->calculate(F, MBPI, MLI);
-#ifndef NDEBUG
if (ViewMachineBlockFreqPropagationDAG != GVDT_None &&
(ViewBlockFreqFuncName.empty() ||
F.getName().equals(ViewBlockFreqFuncName))) {
view();
}
-#endif
return false;
}
@@ -193,13 +189,8 @@ void MachineBlockFrequencyInfo::releaseM
/// rendered using dot.
void MachineBlockFrequencyInfo::view(bool isSimple) const {
// This code is only for debugging.
-#ifndef NDEBUG
ViewGraph(const_cast<MachineBlockFrequencyInfo *>(this),
"MachineBlockFrequencyDAGs", isSimple);
-#else
- errs() << "MachineBlockFrequencyInfo::view is only available in debug builds "
- "on systems with Graphviz or gv!\n";
-#endif // NDEBUG
}
BlockFrequency
Modified: llvm/trunk/lib/CodeGen/MachineBlockPlacement.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/MachineBlockPlacement.cpp?rev=293921&r1=293920&r2=293921&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/MachineBlockPlacement.cpp (original)
+++ llvm/trunk/lib/CodeGen/MachineBlockPlacement.cpp Thu Feb 2 13:18:56 2017
@@ -160,10 +160,8 @@ static cl::opt<unsigned> TailDupPlacemen
extern cl::opt<unsigned> StaticLikelyProb;
extern cl::opt<unsigned> ProfileLikelyProb;
-#ifndef NDEBUG
extern cl::opt<GVDAGType> ViewBlockLayoutWithBFI;
extern cl::opt<std::string> ViewBlockFreqFuncName;
-#endif
namespace {
class BlockChain;
@@ -2365,13 +2363,11 @@ bool MachineBlockPlacement::runOnMachine
MBI->setAlignment(AlignAllNonFallThruBlocks);
}
}
-#ifndef NDEBUG
if (ViewBlockLayoutWithBFI != GVDT_None &&
(ViewBlockFreqFuncName.empty() ||
F->getFunction()->getName().equals(ViewBlockFreqFuncName))) {
MBFI->view(false);
}
-#endif
// We always return true as we have no way to track whether the final order
Modified: llvm/trunk/lib/Transforms/Instrumentation/PGOInstrumentation.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Instrumentation/PGOInstrumentation.cpp?rev=293921&r1=293920&r2=293921&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Instrumentation/PGOInstrumentation.cpp (original)
+++ llvm/trunk/lib/Transforms/Instrumentation/PGOInstrumentation.cpp Thu Feb 2 13:18:56 2017
@@ -1223,7 +1223,6 @@ static bool annotateAllFunctions(
ColdFunctions.push_back(&F);
else if (FreqAttr == PGOUseFunc::FFA_Hot)
HotFunctions.push_back(&F);
-#ifndef NDEBUG
if (PGOViewCounts &&
(PGOViewFunction.empty() || F.getName().equals(PGOViewFunction))) {
LoopInfo LI{DominatorTree(F)};
@@ -1234,7 +1233,6 @@ static bool annotateAllFunctions(
NewBFI->view();
}
-#endif
if (PGOViewRawCounts &&
(PGOViewFunction.empty() || F.getName().equals(PGOViewFunction))) {
if (PGOViewFunction.empty())
More information about the llvm-commits
mailing list