[llvm-dev] Should we switch to --hash-style=both by default in LLD ?

Rui Ueyama via llvm-dev llvm-dev at lists.llvm.org
Mon Oct 2 09:37:10 PDT 2017


Changing default to "both" is probably fine, but what does that change
actually mean in practice? I believe clang already passes --hash-style=both
to the linker by default.

On Mon, Oct 2, 2017 at 9:28 AM, Reid Kleckner via llvm-dev <
llvm-dev at lists.llvm.org> wrote:

> +1 for both table formats. Then we're compatible by default no matter
> what. If somebody cares about .o file size, they can explicitly specify
> their --hash-style.
>
> On Mon, Oct 2, 2017 at 5:35 AM, George Rimar via llvm-dev <
> llvm-dev at lists.llvm.org> wrote:
>
>> >> Le 2 oct. 2017 à 14:23, George Rimar <grimar at accesssoftek.com> a
>> écrit :
>> >>
>> >> I think we can switch LLD either to "both" or probably to "gnu" by
>> default as well.
>> >> Initial version of patch that changes default to "both" is here: D38407
>> >>
>> >> Any thoughts ?
>> >>
>> >> Best regards,
>> >> George | Developer | Access Softek, Inc
>> >
>> >Hi,
>> >
>> >I initially wanted to switch the default to "gnu" as well in GNU
>> linkers. However it was argued that this would generate ELF fiels that do
>> not follow the ELF ABI. See here >https://sourceware.org/ml/bin
>> utils/2017-07/msg00160.html Thus it has been chosen to use "both" rather
>> than "gnu".
>> >
>> >Cheers,
>> >Romain
>>
>> Fair enough, argument that we should not emit ELF files that aren't ABI
>> compliant by default looks reasonable.
>> Use of "both" looks ideal for me then.
>>
>> George.
>> _______________________________________________
>> LLVM Developers mailing list
>> llvm-dev at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev
>>
>
>
> _______________________________________________
> LLVM Developers mailing list
> llvm-dev at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20171002/a1db66c5/attachment.html>


More information about the llvm-dev mailing list