[PATCH] clang-cl: add support for /FA (output assembly file)
Reid Kleckner
rnk at google.com
Tue Oct 15 17:21:26 PDT 2013
LGTM with a fix to make /Fa not require /FA.
================
Comment at: include/clang/Driver/CLCompatOptions.td:118
@@ +117,3 @@
+def _SLASH_Fa : CLJoined<"Fa">,
+ HelpText<"Set filename to use for /FA">,
+ MetaVarName<"<file or directory>">;
----------------
I always use /Fa only, which implies /FA, so I'd make the doc text here stand on its own without referencing /FA.
================
Comment at: test/Driver/cl-outputs.c:106
@@ +105,2 @@
+// RUN: %clang_cl /Fafoo -### -- %s 2>&1 | FileCheck -check-prefix=FaNOFA %s
+// FaNOFA: argument unused during compilation: '/Fafoo'
----------------
This should output asm to foo.asm, I think.
http://llvm-reviews.chandlerc.com/D1946
More information about the cfe-commits
mailing list