<div dir="ltr">Basically we don't want to create more diversity in command line flags that will have to be supported later on, especially for less common flags like these.  -Xclang is really an escape hatch, and -### is needed for testing.<div>
<br></div><div>I could be convinced to add a high-level feature like asan, but then all the documentation out there mentioning -fsanitize=address would be wrong for clang-cl.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Mon, Aug 19, 2013 at 11:47 AM, Jordan Rose <span dir="ltr"><<a href="mailto:jordan_rose@apple.com" target="_blank">jordan_rose@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Out of curiosity, why are you choosing to preserve -Xclang, -### instead of choosing cl.exe-ish names for these? /clang, /###.<br>
<span class="HOEnZb"><font color="#888888"><br>
Jordan<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On Aug 19, 2013, at 11:32 , Hans Wennborg <<a href="mailto:hans@hanshq.net">hans@hanshq.net</a>> wrote:<br>
<br>
> Author: hans<br>
> Date: Mon Aug 19 13:32:59 2013<br>
> New Revision: 188685<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=188685&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=188685&view=rev</a><br>
> Log:<br>
> clang-cl: Expose the -Xclang option<br>
><br>
> Modified:<br>
>    cfe/trunk/include/clang/Driver/Options.td<br>
>    cfe/trunk/test/Driver/cl-options.c<br>
><br>
> Modified: cfe/trunk/include/clang/Driver/Options.td<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/Options.td?rev=188685&r1=188684&r2=188685&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/Options.td?rev=188685&r1=188684&r2=188685&view=diff</a><br>

> ==============================================================================<br>
> --- cfe/trunk/include/clang/Driver/Options.td (original)<br>
> +++ cfe/trunk/include/clang/Driver/Options.td Mon Aug 19 13:32:59 2013<br>
> @@ -264,7 +264,7 @@ def Xassembler : Separate<["-"], "Xassem<br>
>   HelpText<"Pass <arg> to the assembler">, MetaVarName<"<arg>">;<br>
> def Xclang : Separate<["-"], "Xclang">,<br>
>   HelpText<"Pass <arg> to the clang compiler">, MetaVarName<"<arg>">,<br>
> -  Flags<[DriverOption]>;<br>
> +  Flags<[DriverOption, CoreOption]>;<br>
> def Xlinker : Separate<["-"], "Xlinker">, Flags<[LinkerInput, RenderAsInput]>,<br>
>   HelpText<"Pass <arg> to the linker">, MetaVarName<"<arg>">;<br>
> def Xpreprocessor : Separate<["-"], "Xpreprocessor">,<br>
><br>
> Modified: cfe/trunk/test/Driver/cl-options.c<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/cl-options.c?rev=188685&r1=188684&r2=188685&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/cl-options.c?rev=188685&r1=188684&r2=188685&view=diff</a><br>

> ==============================================================================<br>
> --- cfe/trunk/test/Driver/cl-options.c (original)<br>
> +++ cfe/trunk/test/Driver/cl-options.c Mon Aug 19 13:32:59 2013<br>
> @@ -103,3 +103,9 @@<br>
> // RUN: %clang_cl /Zs /Gm /Gm- /GS /Gy /Gy- /GZ -- %s 2>&1<br>
> // RUN: %clang_cl /Zs /RTC1 /wfoo /Zc:wchar_t- -- %s 2>&1<br>
> // RUN: %clang_cl /Zs /ZI /Zi -- %s 2>&1<br>
> +<br>
> +<br>
> +// We support -Xclang for forwarding options to cc1.<br>
> +// RUN: %clang_cl -Xclang hellocc1 -### -- %s 2>&1 | FileCheck -check-prefix=Xclang %s<br>
> +// Xclang: "-cc1"<br>
> +// Xclang: "hellocc1"<br>
><br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div>