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

[Ed Maste via llvm-commits]

emaste added a comment.

> 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.


It's a minor point and probably of no practical consequence so it


================
Comment at: ELF/OutputSections.cpp:415
@@ +414,3 @@
+    // For back-compatibility with old dynamic linkers.
+    WriteVal(DT_SYMBOLIC, 0);
+
----------------
davide wrote:
> 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. 
http://www.sco.com/developers/gabi/latest/ch5.dynamic.html
> Its use has been superseded by the DF_SYMBOLIC flag.


Repository:
  rL LLVM

http://reviews.llvm.org/D13541