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

> 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/binutils/2017-07/msg00160.html Thus it has been chosen
to use "both" rather than "gnu".

Cheers,
Romain

+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/
> binutils/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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20171002/69c3e6f5/attachment.html>