[PATCH] D158689: [llvm-profdata] Use llvm::DenseMap in SampleProfileMap

Thu Aug 31 09:42:52 PDT 2023

ayermolo added a comment.

In D158689#4631799 <https://reviews.llvm.org/D158689#4631799>, @weiwang wrote:

> In D158689#4629229 <https://reviews.llvm.org/D158689#4629229>, @huangjd wrote:
>
>> @weiwang Have you tried to generate a flame graph showing how much time each function takes?
>
> I didn't collect this for the measurement. I think this change is pretty isolated to the loader pass, so if there is a win, it would show up in the e2e compilation.
>
> I'll take a few cases and report back.

and at the end of the day e2e is what we care about, no?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158689