[PATCH] D80487: [ELF] Refine --export-dynamic-symbol semantics to be compatible GNU ld 2.35

Wed May 27 08:38:19 PDT 2020

psmith added a comment.

Looks reasonable to me. Our documentation and help for dynamic-list is a bit sparse, it may be worth emphasising that it implies -bsymbolic (if I've understood correctly).

================
Comment at: lld/ELF/Options.td:148

 defm dynamic_list: Eq<"dynamic-list", "Read a list of dynamic symbols">;

----------------
May be worth adding that this implies -bsymbolic apart from symbols in dynamic-list.

================
Comment at: lld/docs/ld.lld.1:159
 Read a list of dynamic symbols from
 .Ar file .
 .It Fl -eh-frame-hdr
----------------
May be worth adding that this implies -bsymbolic apart from symbols in dynamic-list.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80487/new/

https://reviews.llvm.org/D80487