<div dir="ltr">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.</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 2, 2017 at 9:28 AM, Reid Kleckner via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">+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.</div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Mon, Oct 2, 2017 at 5:35 AM, George Rimar via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><span>>> Le 2 oct. 2017 à 14:23, George Rimar <<a href="mailto:grimar@accesssoftek.com" target="_blank">grimar@accesssoftek.com</a>> a écrit :<br>
>><br>
>> I think we can switch LLD either to "both" or probably to "gnu" by default as well.<br>
>> Initial version of patch that changes default to "both" is here: D38407<br>
>><br>
>> Any thoughts ?<br>
>><br>
>> Best regards,<br>
>> George | Developer | Access Softek, Inc<br>
><br>
</span>>Hi,<br>
><br>
>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 ><a href="https://sourceware.org/ml/binutils/2017-07/msg00160.html" rel="noreferrer" target="_blank">https://sourceware.org/ml/bin<wbr>utils/2017-07/msg00160.html</a> Thus it has been chosen to use "both" rather than "gnu".<br>
><br>
>Cheers,<br>
>Romain<br>
<br>
Fair enough, argument that we should not emit ELF files that aren't ABI compliant by default looks reasonable.<br>
Use of "both" looks ideal for me then.<br>
<span class="m_-3135862067291911247HOEnZb"><font color="#888888"><br>
George.<br>
</font></span></div></div><div class="m_-3135862067291911247HOEnZb"><div class="m_-3135862067291911247h5">______________________________<wbr>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
</div></div></blockquote></div><br></div>
<br>______________________________<wbr>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
<br></blockquote></div><br></div>