[PATCH] D132600: [llvm-profdata] Handle internal linkage functions in profile supplementation

[David Li via Phabricator via llvm-commits]

davidxl added inline comments.


================
Comment at: llvm/tools/llvm-profdata/llvm-profdata.cpp:606
+    } else {
+      StaticFuncMap[NewName] = "---";
+    }
----------------
define a macro for the marker string.


================
Comment at: llvm/tools/llvm-profdata/llvm-profdata.cpp:655
     const sampleprof::FunctionSamples &FS = PD.second;
     auto It = InstrProfileMap.find(FContext.toString());
+    if (It == InstrProfileMap.end()) {
----------------
Skip to the next (using continue) when FS is not interesting (cold etc)


================
Comment at: llvm/tools/llvm-profdata/llvm-profdata.cpp:667
+    }
+    if (FS.getMaxCountInside() > ColdSampleThreshold &&
         It != InstrProfileMap.end() &&
----------------
continue when Itr == end or when Itr is not cold


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

https://reviews.llvm.org/D132600