[clang-tools-extra] [llvm-profdata] Do not create numerical strings for MD5 function names read from a Sample Profile. (PR #66164)

via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 27 15:53:57 PDT 2023


================
@@ -85,7 +86,7 @@ class ProfiledCallGraph {
     std::queue<ContextTrieNode *> Queue;
     for (auto &Child : ContextTracker.getRootContext().getAllChildContext()) {
       ContextTrieNode *Callee = &Child.second;
-      addProfiledFunction(ContextTracker.getFuncNameFor(Callee));
+      addProfiledFunction(Callee->getFuncName());
----------------
WenleiHe wrote:

ProfiledCallGraph needs to build graph with actual function name. This is because it is used to build up processing order for SampleProfileLoader::SymbolMap, which is keyed on string name. 

This change would break it as it changes ProfiledCallGraph to also use MD5. Same for similar changes below.   

https://github.com/llvm/llvm-project/pull/66164


More information about the cfe-commits mailing list