[cfe-commits] [PATCH] Pass -march, -mcpu, -mfpu to linuxtools assembler

[Chandler Carruth]

Sorry for missing this for a while.

--- tools/clang/test/Driver/arm-as.c (revision 0)
+++ tools/clang/test/Driver/arm-as.c (revision 0)

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.

+// RUN: %clang -target arm-linux -### \
+// RUN:   -no-integrated-as -c %s 2>&1 \
+// RUN:   | FileCheck -check-prefix=ARM %s
+// CHECK-ARM: as"

You'll need to handle 'as.exe' here I think.

+// RUN: %clang -target armv7-linux "-mcpu=cortex-a8" -### \

None of the other input lines use "s.

+  Args.AddAllArgs(CmdArgs, options::OPT_march_EQ, options::OPT_mcpu_EQ,
+                  options::OPT_mfpu_EQ);
+

Why add all of the options, shouldn't we just add the last option for these
(unlike -mattr)?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120409/700c5eb6/attachment.html>