[all-commits] [llvm/llvm-project] 482e7d: [ThinLTO] Use DenseMap for OidGuidMap (#107725)

[Kazu Hirata via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 482e7dc67b7de22a47aff63eda1dd719cda86267
      https://github.com/llvm/llvm-project/commit/482e7dc67b7de22a47aff63eda1dd719cda86267
  Author: Kazu Hirata <kazu at google.com>
  Date:   2024-09-07 (Sat, 07 Sep 2024)

  Changed paths:
    M llvm/include/llvm/IR/ModuleSummaryIndex.h

  Log Message:
  -----------
  [ThinLTO] Use DenseMap for OidGuidMap (#107725)

We use OidGuidMap only to map an old GUID to a new one.  We don't use
std::set's strengths like iterators staying valid or the ability to
traverse in a sorted order.

As a data point, during the ThinLTO indexing step of a large
application of ours, we create 440,000 mappings.  Our memory profiler
reports reduction of 127MB in the peak memory usage (out of 4.991GB),
which is bigger than expected savings most likely due to some noise.
The savings should be about 8MB at the max load factor of DenseMap.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications