[cfe-commits] [patch] Default to -std=c++11 on windows
richard at metafoo.co.uk
Wed Jul 11 13:40:03 PDT 2012
On Wed, Jul 11, 2012 at 1:33 PM, Chandler Carruth <chandlerc at google.com>wrote:
> On Wed, Jul 11, 2012 at 1:31 PM, Nico Weber <thakis at chromium.org> wrote:
>> I have another patch locally that changed it there, but the driver has a
>> comment that says "to do: move language stuff from frontend to driver".
>> On Jul 11, 2012 10:21 PM, "Richard Smith" <richard at metafoo.co.uk> wrote:
>>> Is there a reason to put the change there rather than in
>>> CompilerInvocation::setLangDefaults? It seems like this would do the wrong
>>> thing for tools which don't use the driver.
> Put another way, tools which don't use the driver will be quite broken in
> other ways too (builtin header search). The new tooling stuff should always
> round trip through the driver.
I don't think this addresses my main concern. We used to have one place
where the default -std= was set. We now have two, and one of them is not
easy to discover. If we don't have a good reason to split this up, it
should all be handled in the same place.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits