LGTM.

It would be good to document this behavior somewhere.


On Wed, Jul 11, 2012 at 9:21 AM, Nico Weber <[email protected]> wrote:

> Hi,
>
> the attached patch lets clang default to -std=c++11 on windows, as
> discussed on this list previously.
>
> This could be done in either the driver or in the frontend, both
> currently do some language standard processing. A comment in
> lib/Driver/Tools.cpp (2 lines above my patch) suggests that eventually
> all this code should move to the driver, so that's where I put my
> change.
>
> This could also keyed off various things. I settled on comparing the
> triple's OS to llvm::Triple::Win32, since that's what enables
> -fms-compatibility by default too.
>
> Ok?
>
> Nico
>
> _______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to