[PATCH] D107838: [CSSPGO] Do not use getCanonicalFnName in pseudo probe descriptor decoding

Wenlei He via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 10 16:37:59 PDT 2021


wenlei accepted this revision.
wenlei added a comment.

lgtm, thanks.



================
Comment at: llvm/lib/MC/MCPseudoProbe.cpp:360
-    StringRef Name =
-        FunctionSamples::getCanonicalFnName(std::move(*ErrorOrName));
 
----------------
hoy wrote:
> wenlei wrote:
> > does this have `__uniq.`? Can we assert it does not contain `.`?
> It can have that since the uniq suffx is from the frontend. We can assert it doesn't have `.llvm` etc...
ok, as long as it's verified to be no-op now, the risk of something breaking this is low. I think it's fine to leave it as is too.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107838



More information about the llvm-commits mailing list