================
@@ -246,11 +246,27 @@ std::string InstrProfError::message() const {
char InstrProfError::ID = 0;
-std::string getPGOFuncName(StringRef RawFuncName,
- GlobalValue::LinkageTypes Linkage,
+std::string getPGOFuncName(StringRef Name, GlobalValue::LinkageTypes Linkage,
----------------
minglotus-6 wrote:
> Is that still the case?
Clang FE-PGO calls this function, but afaik it's used for coverage rather than
performance nowadays. I guess if Clang FE-PGO continues to use this function it
shouldn't be called `legacy` in its name.
I'm wondering if it makes more sense if I rename `getPGOFuncName` (taking
stringified names) to `getClangPGOFuncName` in this PR? Thoughts?
https://github.com/llvm/llvm-project/pull/74008
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
