[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