<div dir="ltr">LGTM</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jul 19, 2013 at 2:51 PM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Fri, Jul 19, 2013 at 11:39 AM, Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>> wrote:<br>

> +  /// Whether the drvier was invoked as a C++ compiler<br>
> +  /// (e.g. clang++ or clang-cl).<br>
> +  bool CCCIsCXX() const { return Mode == GXXMode || Mode == CLMode; }<br>
><br>
> Looks good, but what needs this?  My understanding is that all of clang, cl,<br>
> and gcc can be invoked on .c or .cpp files and they will auto-detect the<br>
> language from the extension.  The main behavior difference is whether to<br>
> link the standard C++ library.<br>
<br>
</div>I think the mode overrides the auto-detection though, e.g. "clang++<br>
a.c" will treat the file as c++ (and warn about it).<br>
<br>
Checking for CLMode here probably doesn't make sense actually, I'll revert it.<br>
<br>
New patch attached.<br>
<span class="HOEnZb"><font color="#888888"><br>
 - Hans<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
> On Fri, Jul 19, 2013 at 2:31 PM, Hans Wennborg <<a href="mailto:hans@chromium.org">hans@chromium.org</a>> wrote:<br>
>><br>
>> Hi all,<br>
>><br>
>> The attached patch adds a cl.exe compatible mode to the driver. It<br>
>> doesn't actually do anything yet, but the plan is to make it accept<br>
>> cl.exe style command line flags as discussed in [1].<br>
>><br>
>> Please take a look!<br>
>><br>
>> Thanks,<br>
>> Hans<br>
>><br>
>> [1]. <a href="http://lists.cs.uiuc.edu/pipermail/cfe-dev/2013-June/030439.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/cfe-dev/2013-June/030439.html</a><br>
</div></div></blockquote></div><br></div>