[clang] [llvm-profdata] Do not create numerical strings for MD5 function names read from a Sample Profile. (PR #66164)

David Li via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 9 11:38:33 PDT 2023


================
@@ -902,12 +890,16 @@ std::error_code SampleProfileReaderExtBinaryBase::readFuncProfiles() {
       for (const auto &NameOffset : FuncOffsetList) {
         const auto &FContext = NameOffset.first;
         auto FName = FContext.getName();
+        StringRef FNameString;
----------------
david-xl wrote:

auto FName = FContext.GetName() --> it is hard to tell the type of Fname by looking at it. Either using explicit type, or change the getter method name.

https://github.com/llvm/llvm-project/pull/66164


More information about the cfe-commits mailing list