[PATCH] D156569: [InstrProf] Encode linkage names in IRPGO counter names
Ellis Hoag via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 4 11:57:43 PDT 2023
ellis added inline comments.
================
Comment at: llvm/lib/ProfileData/InstrProf.cpp:313
+// See getPGOFuncName()
+std::string getIRPGOFuncName(const Function &F, bool InLTO) {
+ if (!InLTO) {
----------------
MaskRay wrote:
> https://llvm.org/docs/CodingStandards.html#use-namespace-qualifiers-to-implement-previously-declared-functions
Unfortunately most of this file is wrapped in `namespace llvm {}` so this gives a warning
```
warning: extra qualification on member 'getIRPGOFuncName' [-Wextra-qualification]
```
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D156569/new/
https://reviews.llvm.org/D156569
More information about the llvm-commits
mailing list