[all-commits] [llvm/llvm-project] 79cc72: [lld][macho] Strip .__uniq. and .llvm. hashes in -...

[SharonXSharon via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 79cc728b77018bb1d87f0c327f3013aac85ba9fa
      https://github.com/llvm/llvm-project/commit/79cc728b77018bb1d87f0c327f3013aac85ba9fa
  Author: SharonXSharon <xiaoranxu.nju at gmail.com>
  Date:   2025-06-03 (Tue, 03 Jun 2025)

  Changed paths:
    M lld/Common/CMakeLists.txt
    A lld/Common/Utils.cpp
    M lld/ELF/BPSectionOrderer.cpp
    M lld/MachO/BPSectionOrderer.cpp
    M lld/MachO/SectionPriorities.cpp
    M lld/include/lld/Common/BPSectionOrdererBase.inc
    A lld/include/lld/Common/Utils.h
    A lld/test/MachO/order-file-strip-hashes.s

  Log Message:
  -----------
  [lld][macho] Strip .__uniq. and .llvm. hashes in -order_file (#140670)

```
/// Symbols can be appended with "(.__uniq.xxxx)?.llvm.yyyy" where "xxxx" and
/// "yyyy" are numbers that could change between builds. We need to use the root
/// symbol name before this suffix so these symbols can be matched with profiles
/// which may have different suffixes.
```
Just like what we are doing in BP,
https://github.com/llvm/llvm-project/blob/main/lld/MachO/BPSectionOrderer.cpp#L127

the patch removes the suffixes when parsing the order file and getting
the symbol priority to have a better symbol match.

---------

Co-authored-by: Sharon Xu <sharonxu at fb.com>
Co-authored-by: Ellis Hoag <ellis.sparky.hoag at gmail.com>



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