[all-commits] [llvm/llvm-project] e9b9a1: [ELF] Move demoteSymbols to Writer.cpp. NFC

Fangrui Song via All-commits all-commits at lists.llvm.org
Tue Oct 17 13:17:05 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e9b9a1d3202d86d9eb2b49c6463fde0f15f9dc94
      https://github.com/llvm/llvm-project/commit/e9b9a1d3202d86d9eb2b49c6463fde0f15f9dc94
  Author: Fangrui Song <i at maskray.me>
  Date:   2023-10-17 (Tue, 17 Oct 2023)

  Changed paths:
    M lld/ELF/Driver.cpp
    M lld/ELF/Writer.cpp

  Log Message:
  -----------
  [ELF] Move demoteSymbols to Writer.cpp. NFC

History of demoteSharedSymbols:

* https://reviews.llvm.org/D45536 demotes SharedSymbol
* https://reviews.llvm.org/D111365 demotes lazy symbols
* The pending #69295 will demote symbols defined in discarded sections

The pass is placed after markLive just to be clear that it needs `isNeeded`
information computed by markLive. The remaining passes in Driver.cpp do not use
symbol information. Move the pass to Writer.cpp to be closer to other
symbol-related passes.




More information about the All-commits mailing list