Sure. Feel free to directly commit windows documentation patches, post-commit review should be fine there.<div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jul 11, 2012 at 9:47 AM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, Jul 11, 2012 at 6:28 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:<br>

> LGTM.<br>
<br>
Thanks, r160052.<br>
<div class="im"><br>
> It would be good to document this behavior somewhere.<br>
<br>
</div>Patch attached.<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
><br>
> On Wed, Jul 11, 2012 at 9:21 AM, Nico Weber <<a href="mailto:thakis@chromium.org">thakis@chromium.org</a>> wrote:<br>
>><br>
>> Hi,<br>
>><br>
>> the attached patch lets clang default to -std=c++11 on windows, as<br>
>> discussed on this list previously.<br>
>><br>
>> This could be done in either the driver or in the frontend, both<br>
>> currently do some language standard processing. A comment in<br>
>> lib/Driver/Tools.cpp (2 lines above my patch) suggests that eventually<br>
>> all this code should move to the driver, so that's where I put my<br>
>> change.<br>
>><br>
>> This could also keyed off various things. I settled on comparing the<br>
>> triple's OS to llvm::Triple::Win32, since that's what enables<br>
>> -fms-compatibility by default too.<br>
>><br>
>> Ok?<br>
>><br>
>> Nico<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>
</div></div></blockquote></div><br></div>