[llvm] [MC][NFC] Use std::map for AddressProbesMap (PR #99553)

via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 18 22:46:21 PDT 2024


WenleiHe wrote:

> AddressProbesMap is keyed by binary addresses, and it makes sense to treat them as ordered.  

@aaupov this comment doesn't make sense. unordered_map is faster, and it was intentional. Order wasn't needed.

The intention of this patch is only obvious with the other patch that uses the map, so it'd be better to merge the two in one patch. 

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


More information about the llvm-commits mailing list