[all-commits] [llvm/llvm-project] 3733ed: [ELF] Introduce Symbol::isExported to cache includ...

[Fangrui Song via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 3733ed6f1c6b0eef1e13e175ac81ad309fc0b080
      https://github.com/llvm/llvm-project/commit/3733ed6f1c6b0eef1e13e175ac81ad309fc0b080
  Author: Fangrui Song <i at maskray.me>
  Date:   2024-12-08 (Sun, 08 Dec 2024)

  Changed paths:
    M lld/ELF/Driver.cpp
    M lld/ELF/LTO.cpp
    M lld/ELF/MarkLive.cpp
    M lld/ELF/Relocations.cpp
    M lld/ELF/Symbols.cpp
    M lld/ELF/Symbols.h
    M lld/ELF/Writer.cpp

  Log Message:
  -----------
  [ELF] Introduce Symbol::isExported to cache includeInDynsym

isExported, intended to replace exportDynamic, is primarily set in two
locations, (a) after parseSymbolVersion and (b) during demoteSymbols.

In the future, we should try removing exportDynamic. Currently,
merging exportDynamic/isExported would cause
riscv-gp.s to fail:

* The first isExported computation considers the undefined symbol exported
* Defined as a linker-synthesized symbol
* isExported remains true, while it should be false



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