<div dir="ltr">We could add a "-lld-" prefix or something like that to new options that we add to LLD, but it reminds me of web browser developers who used to be habitually adding "-webkit-", "-moz-" or "-ms-" prefixes to new CSS properties. They've now reached the conclusion that that was not a good idea; until properties are standardized web developers had to list the same properties with all these different prefixes, and once they are standardized, they've got to add one more to the list without prefix. That was crazy.<div><br></div><div>CSS property names are different from command line options, but the situation is similar, so as long as a new option has a chance to make sense with other linkers, I think we should just give a generic name to the option.</div><div><div><div><div><div class="gmail_extra"><br></div><div class="gmail_extra">I'm wondering why you think users shouldn't use --lto-newpm-passes or -mllvm. As long as they understand the risk of using it, that should be fine, no?</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 26, 2016 at 10:25 PM, Sean Silva <span dir="ltr"><<a href="mailto:chisophugis@gmail.com" target="_blank">chisophugis@gmail.com</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">This is for compatibilty with gold. <a href="http://manpages.ubuntu.com/manpages/precise/man1/ld.1.html" target="_blank">http://manpages.ubuntu.com/manpages/precise/man1/ld.1.html</a><div><br></div><div>But this does point out something important: we don't have a clear delineation between options that are LLD-specific and ones that are for compatibility.</div><div><br></div><div>Rui, what do you think about adopting some convention about lld-specific options? Some of them are quite low level (like --lto-newpm-passes or -mllvm) and I don't think we want users to be relying on them. This is especially important as LLD is getting close to being let out in the wild in FreeBSD and users might start relying on the options.<span class="HOEnZb"><font color="#888888"><div><br></div><div>-- Sean Silva</div></font></span></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 26, 2016 at 10:01 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</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">Perhaps renaming it to --multithreaded? or --parallel? (or some toher similar name - perhaps there's precedent/inspiration in the existing flag name for similar behavior in other (LLVM or non-LLVM) tools?)</div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 26, 2016 at 9:30 PM, Sean Silva 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">Author: silvas<br>
Date: Thu May 26 23:30:27 2016<br>
New Revision: 270963<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=270963&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=270963&view=rev</a><br>
Log:<br>
--threads is a flag, not a number<br>
<br>
We would previously accept `--threads=4`, but this option just turns on<br>
threading and does not specify a number of threads.<br>
<br>
I ran into this by accident because I was passing `--threads=<n>` but<br>
the number didn't seem to affect anything.<br>
<br>
Modified:<br>
    lld/trunk/ELF/Options.td<br>
<br>
Modified: lld/trunk/ELF/Options.td<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/Options.td?rev=270963&r1=270962&r2=270963&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/Options.td?rev=270963&r1=270962&r2=270963&view=diff</a><br>
==============================================================================<br>
--- lld/trunk/ELF/Options.td (original)<br>
+++ lld/trunk/ELF/Options.td Thu May 26 23:30:27 2016<br>
@@ -150,7 +150,7 @@ def strip_debug : Flag<["--"], "strip-de<br>
 def sysroot : Joined<["--"], "sysroot=">,<br>
   HelpText<"Set the system root">;<br>
<br>
-def threads : Joined<["--"], "threads">;<br>
+def threads : Flag<["--"], "threads">;<br>
<br>
 def trace: Flag<["--"], "trace">,<br>
   HelpText<"Print the names of the input files">;<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">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>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div></div></div></div></div></div>