[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