================ @@ -519,13 +591,20 @@ class InstrProfSymtab { /// will be represented using the same StringRef value. inline StringRef getFuncNameOrExternalSymbol(uint64_t FuncMD5Hash); + /// Just like getFuncName, except that it will return a non-empty StringRef ---------------- minglotus-6 wrote:
Cleaned this up. I just realized `getFuncNameOrExternalSymbol` and `getFuncName` (line 587) needs a rename. These two functions are used in many places (llvm-profdata, text profile writer, coverage mapping, and unit test). I'm going to send out an NFC for the rename. https://github.com/llvm/llvm-project/pull/66825 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits