<div class="gmail_quote">On Mon, Apr 9, 2012 at 2:41 PM, Evgeniy Stepanov <span dir="ltr"><<a href="mailto:eugeni.stepanov@gmail.com">eugeni.stepanov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Thanks! All good points. New patch attached.<br></blockquote><div><br></div><div>LGTM.</div><div><br></div><div>Feel free to submit and then address the ABI thing Anton mentioned. I like his idea.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="HOEnZb"><div class="h5"><br>
On Mon, Apr 9, 2012 at 3:00 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:<br>
> Sorry for missing this for a while.<br>
><br>
> --- tools/clang/test/Driver/arm-as.c (revision 0)<br>
> +++ tools/clang/test/Driver/arm-as.c (revision 0)<br>
><br>
> Your change and these tests are Linux-specific, so we should put these in<br>
> 'linux-as.c' or 'linux-arm-as.c'. I'm fine with either.<br>
><br>
> +// RUN: %clang -target arm-linux -### \<br>
> +// RUN:   -no-integrated-as -c %s 2>&1 \<br>
> +// RUN:   | FileCheck -check-prefix=ARM %s<br>
> +// CHECK-ARM: as"<br>
><br>
> You'll need to handle 'as.exe' here I think.<br>
><br>
> +// RUN: %clang -target armv7-linux "-mcpu=cortex-a8" -### \<br>
><br>
> None of the other input lines use "s.<br>
><br>
> +  Args.AddAllArgs(CmdArgs, options::OPT_march_EQ, options::OPT_mcpu_EQ,<br>
> +                  options::OPT_mfpu_EQ);<br>
> +<br>
><br>
> Why add all of the options, shouldn't we just add the last option for these<br>
> (unlike -mattr)?<br>
</div></div></blockquote></div><br>