[all-commits] [llvm/llvm-project] 3b8b10: [MC] Make ELFEntrySizeMap a DenseMap (#91728)

aengelke via All-commits all-commits at lists.llvm.org
Tue May 14 02:39:00 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 3b8b1022684175e988f043f14596f2dc9b31c6c7
      https://github.com/llvm/llvm-project/commit/3b8b1022684175e988f043f14596f2dc9b31c6c7
  Author: aengelke <engelke at in.tum.de>
  Date:   2024-05-14 (Tue, 14 May 2024)

  Changed paths:
    M llvm/include/llvm/MC/MCContext.h
    M llvm/lib/MC/MCContext.cpp

  Log Message:
  -----------
  [MC] Make ELFEntrySizeMap a DenseMap (#91728)

There is no need for an ordered std::map and also no need to duplicate
the section name, which is owned by the ELFSectionKey. Therefore, use a
DenseMap instead and don't copy the string. As a further, minor
performance optimization, avoid the hash table lookup in
isELFGenericMergeableSection when the section name was just added.

This slightly improves compilation performance in our application, where
we occasionally compile many small object files.



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


More information about the All-commits mailing list