This is an archive of the discontinued LLVM Phabricator instance.

Differential D21574

Add an internal option to specify the function of interest to display MBFI
ClosedPublic

Authored by davidxl on Jun 21 2016, 4:49 PM.

Details

Reviewers
silvas
danielcdh

Diff Detail

Event Timeline

davidxl updated this revision to Diff 61461.Jun 21 2016, 4:49 PM
davidxl retitled this revision from to Add an internal option to specify the function of interest to display MBFI.
davidxl updated this object.
davidxl added reviewers: danielcdh, silvas.
davidxl added a subscriber: llvm-commits.
silvas accepted this revision.Jun 21 2016, 5:43 PM
silvas edited edge metadata.

LGTM. This seems handy!

This revision is now accepted and ready to land.Jun 21 2016, 5:43 PM
danielcdh accepted this revision.Jun 21 2016, 6:31 PM
danielcdh edited edge metadata.
Eugene.Zelenko added a subscriber: Eugene.Zelenko.Sep 22 2016, 5:21 PM

Committed in r273366. Please specify Differential revision in commit message.

Eugene.Zelenko closed this revision.Sep 22 2016, 5:21 PM

Committed in r273366. Please specify Differential revision in commit message.

Revision Contents

PathSize
lib/
CodeGen/
7 lines

Diff 61461

lib/CodeGen/MachineBlockFrequencyInfo.cpp

Show All 36 Lines cl::values(clEnumValN(GVDT_None, "none", "do not display graphs."),
clEnumValN(GVDT_Fraction, "fraction", clEnumValN(GVDT_Fraction, "fraction",
"display a graph using the " "display a graph using the "
"fractional block frequency representation."), "fractional block frequency representation."),
clEnumValN(GVDT_Integer, "integer", clEnumValN(GVDT_Integer, "integer",
"display a graph using the raw " "display a graph using the raw "
"integer fractional block frequency representation."), "integer fractional block frequency representation."),
clEnumValEnd)); clEnumValEnd));
static cl::opt<std::string> ViewMachineBlockFreqFuncName("view-mbfi-func-name",
cl::Hidden);
namespace llvm { namespace llvm {
template <> struct GraphTraits<MachineBlockFrequencyInfo *> { template <> struct GraphTraits<MachineBlockFrequencyInfo *> {
typedef const MachineBasicBlock NodeType; typedef const MachineBasicBlock NodeType;
typedef MachineBasicBlock::const_succ_iterator ChildIteratorType; typedef MachineBasicBlock::const_succ_iterator ChildIteratorType;
typedef MachineFunction::const_iterator nodes_iterator; typedef MachineFunction::const_iterator nodes_iterator;
static inline const NodeType * static inline const NodeType *
▲ Show 20 Lines • Show All 79 Lines • ▼ Show 20 Lines
bool MachineBlockFrequencyInfo::runOnMachineFunction(MachineFunction &F) { bool MachineBlockFrequencyInfo::runOnMachineFunction(MachineFunction &F) {
MachineBranchProbabilityInfo &MBPI = MachineBranchProbabilityInfo &MBPI =
getAnalysis<MachineBranchProbabilityInfo>(); getAnalysis<MachineBranchProbabilityInfo>();
MachineLoopInfo &MLI = getAnalysis<MachineLoopInfo>(); MachineLoopInfo &MLI = getAnalysis<MachineLoopInfo>();
if (!MBFI) if (!MBFI)
MBFI.reset(new ImplType); MBFI.reset(new ImplType);
MBFI->calculate(F, MBPI, MLI); MBFI->calculate(F, MBPI, MLI);
#ifndef NDEBUG #ifndef NDEBUG
if (ViewMachineBlockFreqPropagationDAG != GVDT_None) { if (ViewMachineBlockFreqPropagationDAG != GVDT_None &&
(ViewMachineBlockFreqFuncName.empty() ||
F.getName().equals(ViewMachineBlockFreqFuncName))) {
view(); view();
} }
#endif #endif
return false; return false;
} }
void MachineBlockFrequencyInfo::releaseMemory() { MBFI.reset(); } void MachineBlockFrequencyInfo::releaseMemory() { MBFI.reset(); }
Show All 37 Lines