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

[Davide Italiano via llvm-commits]

On Thu, Oct 8, 2015 at 2:42 AM, Joerg Sonnenberger via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
> 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.
>

It's optional for shared objects and ignored for executables, d_un.
ELF specification, version 1.1, 2-12.

-- 
Davide

"There are no solved problems; there are only problems that are more
or less solved" -- Henri Poincare