<div dir="ltr">And as a side note, if we want to add command line options that are just ignored, we don't want to add help texts just to say that they are ignored. One day when we have a man page, we would want to write about that in it, but they probably don't worth to take the real estate in the --help message.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 23, 2015 at 11:41 AM, George Rimar via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@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">Ok, its sounds reasonable. Sorry for this patch. Please revert then - I dont have write permissions for that.<br>
<span class="">________________________________________<br>
От: <a href="mailto:davide.italiano@gmail.com">davide.italiano@gmail.com</a> [<a href="mailto:davide.italiano@gmail.com">davide.italiano@gmail.com</a>] от имени Davide Italiano [<a href="mailto:davide@freebsd.org">davide@freebsd.org</a>]<br>
</span>Отправлено: 23 сентября 2015 г. 22:35<br>
Кому: George Rimar<br>
Копия: Rafael Espíndola; llvm-commits<br>
<span class="im HOEnZb">Тема: Re: [lld] r248389 - [ELF2] - added ignored command line options for compatibility.<br>
<br>
</span><span class="im HOEnZb">On Wed, Sep 23, 2015 at 11:05 AM, George Rimar <<a href="mailto:grimar@accesssoftek.com">grimar@accesssoftek.com</a><mailto:<a href="mailto:grimar@accesssoftek.com">grimar@accesssoftek.com</a>>> wrote:<br>
> I found the description of that flag in The GNU Linker book - <a href="http://www.eecs.umich.edu/courses/eecs373/readings/Linker.pdf" rel="noreferrer" target="_blank">http://www.eecs.umich.edu/courses/eecs373/readings/Linker.pdf</a><br>
> Page 14:<br>
> -assert keyword<br>
> This option is ignored for SunOS compatibility<br>
<br>
This was insteresting. Unfortunately, option handling doesn't seem to be very consistent between version and documentation.<br>
<br>
On binutils 2.25:<br>
% /usr/local/bin/ld.bfd -assert<br>
/usr/local/bin/ld.bfd: unrecognized option '-assert'<br>
<br>
% /usr/local/bin/ld.gold -assert<br>
/usr/local/bin/ld.gold: -assert: missing argument<br>
<br>
It seems that also older versions of ld don't ignore that version, just not recognize it (both single and double dash).<br>
<br>
When there's ambiguity, I generally err on the side of being a little bit stricter (unless there's a good reason not to).<br>
In this case in particular, it seems you don't have a real use case for this, and there's some contraddiction between what documentation says, what ld.bfd does and what gold does.<br>
<br>
At the light of these facts, I propose to revert this given it seems whatever choice you make you violate something, and given this option has probably impact on a negligible amount of users.<br>
<br>
Please let me know if you have any objections.<br>
<br>
Thanks,<br>
<br>
--<br>
Davide<br>
<br>
"There are no solved problems; there are only problems that are more or less solved" -- Henri Poincare<br>
</span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>