On Wed, Jul 11, 2012 at 1:40 PM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank" class="cremed">richard@metafoo.co.uk</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><div class="gmail_quote">On Wed, Jul 11, 2012 at 1:33 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank" class="cremed">chandlerc@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>On Wed, Jul 11, 2012 at 1:31 PM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank" class="cremed">thakis@chromium.org</a>></span> wrote:<br></div><div class="gmail_extra"><div class="gmail_quote">

<div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">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".</p>


<div><div>
<div class="gmail_quote">On Jul 11, 2012 10:21 PM, "Richard Smith" <<a href="mailto:richard@metafoo.co.uk" target="_blank" class="cremed">richard@metafoo.co.uk</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



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.<br></blockquote></div></div></div></blockquote>


<div><br></div></div><div>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.</div></div>

</div>
</blockquote></div><br></div></div><div>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.</div>

</blockquote></div><br></div><div class="gmail_extra">The reason is to migrate all of them to the driver.</div><div class="gmail_extra"><br></div><div class="gmail_extra">There is already logic for handling -std= in the driver that would be relevant to setting the default as well.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">I would like there to be no further work on in-Frontend flag management, and instead focus on moving it to the driver. I'm happy to see the rest of that routine move the driver too, as then I agree we will have only one place responsible for this.</div>