[PATCH] D18623: [PGO] refactor PGOFuncName meta data code to be used in clang
David Li via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 30 14:32:40 PDT 2016
davidxl added inline comments.
================
Comment at: include/llvm/ProfileData/InstrProf.h:250
@@ -247,1 +249,3 @@
+inline StringRef PGOFuncNameMetaData(){
+ return "PGOFuncName";
----------------
clang-format it.
-->getPGOFuncNameMetaDataKind()
================
Comment at: include/llvm/ProfileData/InstrProf.h:262
@@ +261,3 @@
+/// only. Return true if the meta data is written out, and false if not.
+inline bool createPGOFuncNameMetaData(Function &F) {
+ const std::string &FuncName = getPGOFuncName(F);
----------------
Not suitable for being an 'inline' function. Put the def in InstrProf.cpp.
http://reviews.llvm.org/D18623
More information about the llvm-commits
mailing list