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

[Wei Wang via Phabricator via llvm-commits]

weiwang added a comment.

In D158689#4628952 <https://reviews.llvm.org/D158689#4628952>, @wenlei wrote:

>> Pick the top 100 slow compilations from a major internal build target.
>
> Among the 100 compilations measured, is there anything that shows a meaningful improvement from DenseMap?

Not really. For instruction count, the biggest win from DenseMap is 0.14%, and unordered_map also wins by 0.17% in one case. For wall clock time, there is a bigger swing due to bigger variance. DenseMap wins as much as 4.3%, but unordered_map also shows comparable wins in other cases.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158689