[PATCH] D29975: Include func name in dot file name.
David Li via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 14 17:21:59 PST 2017
davidxl created this revision.
For better debugging experience.
https://reviews.llvm.org/D29975
Files:
include/llvm/CodeGen/MachineBlockFrequencyInfo.h
lib/CodeGen/BranchFolding.cpp
lib/CodeGen/BranchFolding.h
lib/CodeGen/MachineBlockFrequencyInfo.cpp
lib/CodeGen/MachineBlockPlacement.cpp
Index: lib/CodeGen/MachineBlockPlacement.cpp
===================================================================
--- lib/CodeGen/MachineBlockPlacement.cpp
+++ lib/CodeGen/MachineBlockPlacement.cpp
@@ -2381,7 +2381,7 @@
if (ViewBlockLayoutWithBFI != GVDT_None &&
(ViewBlockFreqFuncName.empty() ||
F->getFunction()->getName().equals(ViewBlockFreqFuncName))) {
- MBFI->view(false);
+ MBFI->view("MBP." + MF.getName(), false);
}
Index: lib/CodeGen/MachineBlockFrequencyInfo.cpp
===================================================================
--- lib/CodeGen/MachineBlockFrequencyInfo.cpp
+++ lib/CodeGen/MachineBlockFrequencyInfo.cpp
@@ -181,7 +181,7 @@
if (ViewMachineBlockFreqPropagationDAG != GVDT_None &&
(ViewBlockFreqFuncName.empty() ||
F.getName().equals(ViewBlockFreqFuncName))) {
- view();
+ view("MachineBlockFrequencyDAGS." + F.getName());
}
}
@@ -197,10 +197,9 @@
/// Pop up a ghostview window with the current block frequency propagation
/// rendered using dot.
-void MachineBlockFrequencyInfo::view(bool isSimple) const {
-// This code is only for debugging.
- ViewGraph(const_cast<MachineBlockFrequencyInfo *>(this),
- "MachineBlockFrequencyDAGs", isSimple);
+void MachineBlockFrequencyInfo::view(const Twine &Name, bool isSimple) const {
+ // This code is only for debugging.
+ ViewGraph(const_cast<MachineBlockFrequencyInfo *>(this), Name, isSimple);
}
BlockFrequency
Index: lib/CodeGen/BranchFolding.h
===================================================================
--- lib/CodeGen/BranchFolding.h
+++ lib/CodeGen/BranchFolding.h
@@ -122,7 +122,7 @@
const MachineBasicBlock *MBB) const;
raw_ostream &printBlockFreq(raw_ostream &OS,
const BlockFrequency Freq) const;
- void view(bool isSimple = true);
+ void view(const Twine &Name, bool isSimple = true);
uint64_t getEntryFreq() const;
private:
Index: lib/CodeGen/BranchFolding.cpp
===================================================================
--- lib/CodeGen/BranchFolding.cpp
+++ lib/CodeGen/BranchFolding.cpp
@@ -497,7 +497,9 @@
return MBFI.printBlockFreq(OS, Freq);
}
-void BranchFolder::MBFIWrapper::view(bool isSimple) { MBFI.view(isSimple); }
+void BranchFolder::MBFIWrapper::view(const Twine &Name, bool isSimple) {
+ MBFI.view(Name, isSimple);
+}
uint64_t
BranchFolder::MBFIWrapper::getEntryFreq() const {
Index: include/llvm/CodeGen/MachineBlockFrequencyInfo.h
===================================================================
--- include/llvm/CodeGen/MachineBlockFrequencyInfo.h
+++ include/llvm/CodeGen/MachineBlockFrequencyInfo.h
@@ -62,7 +62,7 @@
const MachineFunction *getFunction() const;
const MachineBranchProbabilityInfo *getMBPI() const;
- void view(bool isSimple = true) const;
+ void view(const Twine& Name, bool isSimple = true) const;
// Print the block frequency Freq to OS using the current functions entry
// frequency to convert freq into a relative decimal form.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D29975.88470.patch
Type: text/x-patch
Size: 3085 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170215/18d2fa5b/attachment.bin>
More information about the llvm-commits
mailing list