[PATCH] D29975: Include func name in dot file name.

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 15 10:48:31 PST 2017


vsk accepted this revision.
vsk added a comment.
This revision is now accepted and ready to land.

LGTM with a minor nitpick.



================
Comment at: include/llvm/CodeGen/MachineBlockFrequencyInfo.h:65
   const MachineBranchProbabilityInfo *getMBPI() const;
-  void view(bool isSimple = true) const;
+  void view(const Twine& Name, bool isSimple = true) const;
 
----------------
nit, formatting


https://reviews.llvm.org/D29975





More information about the llvm-commits mailing list