[PATCH] D13541: [ELF2] Add support for -Bsymbolic
Joerg Sonnenberger via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 8 02:42:28 PDT 2015
On Thu, Oct 08, 2015 at 12:28:48AM +0000, Ed Maste via llvm-commits wrote:
> emaste added inline comments.
>
> ================
> Comment at: ELF/OutputSections.cpp:415
> @@ +414,3 @@
> + // For back-compatibility with old dynamic linkers.
> + WriteVal(DT_SYMBOLIC, 0);
> +
> ----------------
> Are there relevant rtlds that require DT_SYMBOLIC?
What do you mean with "require"? I think the comment tis just wrong,
when using -Bsymbolic the DT_SYMBOLIC entry is necessary as it changes
the behavior of the linker? It's not about compatibility either.
Joerg
More information about the llvm-commits
mailing list