[llvm-branch-commits] [llvm] [BOLT] Match blocks with calls as anchors (PR #96596)

Shaw Young via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Wed Jul 3 09:19:16 PDT 2024


================
@@ -479,6 +481,11 @@ Error YAMLProfileReader::readProfile(BinaryContext &BC) {
   NormalizeByInsnCount = usesEvent("cycles") || usesEvent("instructions");
   NormalizeByCalls = usesEvent("branches");
 
+  // Map profiled function ids to names.
+  DenseMap<uint32_t, std::string *> IdToFunctionName;
----------------
shawbyoung wrote:

I came across an edge case where using a vector would not work - a testcase where a profile had 16 profiled function but the largest profile id was 78. Using a vector, e.g. resizing it to the # of profiled functions and using function ids to index into the vector, would not work in this case.

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


More information about the llvm-branch-commits mailing list