<div>Sorry for missing this for a while.</div><div><br></div><div><div>--- tools/clang/test/Driver/arm-as.c<span class="Apple-tab-span" style="white-space:pre">    </span>(revision 0)</div><div>+++ tools/clang/test/Driver/arm-as.c<span class="Apple-tab-span" style="white-space:pre">     </span>(revision 0)</div>
</div><div><br></div><div>Your change and these tests are Linux-specific, so we should put these in 'linux-as.c' or 'linux-arm-as.c'. I'm fine with either.</div><div><br></div><div><div>+// RUN: %clang -target arm-linux -### \</div>
<div>+// RUN:   -no-integrated-as -c %s 2>&1 \</div><div>+// RUN:   | FileCheck -check-prefix=ARM %s</div><div>+// CHECK-ARM: as"</div></div><div><br></div><div>You'll need to handle 'as.exe' here I think.</div>
<div><br></div><div><div>+// RUN: %clang -target armv7-linux "-mcpu=cortex-a8" -### \</div></div><div><br></div><div>None of the other input lines use "s.</div><div><br></div><div><div>+  Args.AddAllArgs(CmdArgs, options::OPT_march_EQ, options::OPT_mcpu_EQ,</div>
<div>+                  options::OPT_mfpu_EQ);</div><div>+</div></div><div><br></div><div>Why add all of the options, shouldn't we just add the last option for these (unlike -mattr)?</div>