[PATCH] D98226: [CSSPGO][llvm-profgen] Fix getCanonicalFnName usage in llvm-profgen

Lei Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 9 12:14:58 PST 2021


wlei added inline comments.


================
Comment at: llvm/tools/llvm-profgen/PseudoProbe.cpp:165
     uint32_t NameSize = readUnsignedNumber<uint32_t>();
-    StringRef Name = readString(NameSize);
+    StringRef Name = FunctionSamples::getCanonicalFnName(readString(NameSize));
 
----------------
hoy wrote:
> Give it "selected" to avoid dropping .__unique suffix?
The "selected" is already the default parameter, you can find it in https://reviews.llvm.org/D96932.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D98226/new/

https://reviews.llvm.org/D98226



More information about the llvm-commits mailing list