[PATCH] D158322: [ELF] Add -Bsymbolic-non-weak

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 18 16:19:59 PDT 2023


MaskRay added a comment.

Glad to know see more adoption of the `-Bsymbolic-non-weak*` idea:) (https://maskray.me/blog/2021-05-16-elf-interposition-and-bsymbolic)

> We have use cases for which making weak data preemptible is useful, but we don't want to pessimize access to non-weak data. (For a large internal app, we measured 2000+ data symbols whose accesses would be unnecessarily pessimized by -Bsymbolic-functions.)

The main problem is copy relocations, which is incompatible with making non-weak data non-preemptible.
Are you able to rule out all copy relocations  (default ELF codegen for -fno-pic)?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158322



More information about the llvm-commits mailing list