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 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"><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 class="HOEnZb"><div class="h5">
<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>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>