[PATCH] D13541: [ELF2] Add support for -Bsymbolic

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 7 17:35:15 PDT 2015


davide added inline comments.

================
Comment at: ELF/OutputSections.cpp:415
@@ +414,3 @@
+    // For back-compatibility with old dynamic linkers.
+    WriteVal(DT_SYMBOLIC, 0);
+
----------------
emaste wrote:
> Are there relevant rtlds that require DT_SYMBOLIC?
Not that I'm aware of. That said, I mainly did it to match gold behavior.
If you think it's not required, I can drop it, although this comes almost for free, so I'm not that inclined to drop it. 


Repository:
  rL LLVM

http://reviews.llvm.org/D13541





More information about the llvm-commits mailing list