<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 17, 2016 at 5:13 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On 16 October 2016 at 22:13, Davide Italiano <<a href="mailto:davide@freebsd.org" target="_blank">davide@freebsd.org</a>> wrote:<br>
> On Sun, Oct 16, 2016 at 6:43 PM, Sean Silva <<a href="mailto:chisophugis@gmail.com" target="_blank">chisophugis@gmail.com</a>> wrote:<br>
>> Nice to see this land!<br>
>><br>
>> One nit:<br>
>> Currently, doesn't LLD/ELF ignore -plugin-opt? That will mean that if a user<br>
>> uses the "gold syntax" then LLD will silently ignore it, which isn't good.<br>
>> At the very least, can we issue an error if we see `-plugin-opt jobs=N` and<br>
>> suggest the LLD spelling?<br>
>><br>
>> Or maybe just accept the gold syntax? Our current handling of `-plugin` and<br>
>> `-plugin-opt` is intended to make LLD transparently Do The Right Thing when<br>
>> LLD is invoked as if it were gold, so clearly gold compatibility is<br>
>> important enough for that. This suggests it is important enough to be<br>
>> compatible from a ThinLTO perspective too.<br>
>><br>
><br>
> I agree with what you're suggesting.  My initial vote would be for<br>
> error'ing out on anything we can't understand that's passed via<br>
> `-plugin-opt` and see what breaks (and add incremental support for<br>
> every feature needed).<br>
> Teresa, Rafael, any opinions about it?<br>
<br>
</span>I agree. Having clang known if it is using gold or lld is probably not worth it.<br></blockquote><div><br></div><div>Sure, that seems reasonable to me as well. For example, there is now a clang option-flto-jobs=N that hooks up to the gold plugin option jobs=N option, and you would get that automatically without having to wire it in. Erroring on any unrecognized options would be good too.</div><div><br></div><div>Teresa</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="m_6146622078711417165gmail_signature" data-smartmail="gmail_signature"><span style="font-family:Times;font-size:medium"><table cellspacing="0" cellpadding="0"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small"><td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Teresa Johnson |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Software Engineer |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a> |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"> <a href="tel:408-460-2413" value="+14084602413" target="_blank">408-460-2413</a></td></tr></tbody></table></span></div>
</div></div>