================
@@ -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
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to