[PATCH] D156569: [InstrProf] Encode linkage names in IRPGO counter names

[David Li via Phabricator via llvm-commits]

davidxl added inline comments.


================
Comment at: llvm/lib/ProfileData/InstrProf.cpp:271
+// name. The filepath is used to discriminate possibly identical function names.
+// ; is used because it is unlikely to be found in either <filepath> or
+// <linkage-name>.
----------------
ellis wrote:
> davidxl wrote:
> > is it better to use '>' to indicate context:
> > 
> > foo.c > bar
> > 
> > or <foo.c> bar
> I don't think `>` would work well because objc_direct methods have `<>` symbols: `-<C foo:>`. I thought `;` worked well because I don't know of any name mangling that uses it.
ok. Note the <> is used in the unknown case.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156569