On Tuesday, October 6, 2015, Ed Maste <<a href="mailto:emaste@freebsd.org">emaste@freebsd.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 4 October 2015 at 14:40, Saleem Abdulrasool <<a href="javascript:;" onclick="_e(event, 'cvml', 'compnerd@compnerd.org')">compnerd@compnerd.org</a>> wrote:<br>
> On Fri, Oct 2, 2015 at 11:54 PM, Ed Maste via llvm-commits<br>
> <<a href="javascript:;" onclick="_e(event, 'cvml', 'llvm-commits@lists.llvm.org')">llvm-commits@lists.llvm.org</a>> wrote:<br>
>><br>
>> Author: emaste<br>
>> Date: Sat Oct  3 01:54:24 2015<br>
>> New Revision: 249230<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=249230&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=249230&view=rev</a><br>
>> Log:<br>
>> Add -rpath= alias for -rpath<br>
>><br>
>> Modified:<br>
>>     lld/trunk/ELF/Options.td<br>
>><br>
>> Modified: lld/trunk/ELF/Options.td<br>
>> URL:<br>
>> <a href="http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/Options.td?rev=249230&r1=249229&r2=249230&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/Options.td?rev=249230&r1=249229&r2=249230&view=diff</a><br>
>><br>
>> ==============================================================================<br>
>> --- lld/trunk/ELF/Options.td (original)<br>
>> +++ lld/trunk/ELF/Options.td Sat Oct  3 01:54:24 2015<br>
>> @@ -75,6 +75,7 @@ def alias_discard_all_x: Flag<["-"], "x"<br>
>>  def alias_discard_locals_X: Flag<["-"], "X">, Alias<discard_locals>;<br>
>>  def alias_entry_e : Separate<["-"], "e">, Alias<entry>;<br>
>>  def alias_l__library : Joined<["--"], "library=">, Alias<l>;<br>
>> +def alias_rpath_rpath : Joined<["-"], "rpath=">, Alias<rpath>;<br>
>>  def alias_soname_h : Separate<["-"], "h">, Alias<soname>;<br>
>>  def alias_soname_soname : Separate<["-"], "soname">, Alias<soname>;<br>
><br>
><br>
> Couldn't we use JoinedOrSeparate as we do elsewhere in clang for the rpath<br>
> alias here?  It really is that -rpath may be joined or separate.<br>
<br>
That seems to be the case (in GNU ld) for all of these options.<br>
However, an = is necessary for the joined form, and as far as I can<br>
tell JoinedOrSeparate doesn't handle that. That is, -rpathfoo would<br>
also be accepted. JoinedWithEqualsOrSeparate?<br>
</blockquote><div><br></div><div>Yeah, that would work.  The idea is that many arguments are going to be of this form, so might as well as make it easy.<br></div><br><br>-- <br>Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org<br>