<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Aug 24, 2015 at 5:50 PM, Eric Christopher via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-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">echristo added inline comments.<br>
<span class=""><br>
================<br>
Comment at: include/clang/Driver/Options.td:1853<br>
@@ -1853,1 +1852,3 @@<br>
</span><span class="">+def fuse_ld_EQ : Joined<["-"], "fuse-ld=">, HelpText<"Use linker <name>">, Group<f_Group>;<br>
</span>+def linker_EQ : Joined<["-"], "linker=">, Alias<fuse_ld_EQ>, MetaVarName<"<name>">;<br>
<br>
----------------<br>
<span class="">thakis wrote:<br>
> Any reason to have another alias for this at all? Does gcc understand this spelling? If not, could you limit this flag to PS4 targets? (I'm guessing you need it for PS4 targets for some reason.)<br>
</span>Any reason? (And I'm not sure how to limit it btw on target).</blockquote><div><br></div><div>-l already has a meaning; adding a different flag starting with -l is a bad idea.</div></div></div></div>