<div dir="ltr">LGTM</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Oct 29, 2013 at 7:48 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ping<br>
<div class="HOEnZb"><div class="h5"><br>
On 25 September 2013 17:44, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
> With this patch -Wwrite-strings is still implemented with the terrible<br>
> hack of passing -fconst-strings to -cc1, but at least the driver uses<br>
> the regular warning checking code path.<br>
><br>
> Since we don't support a warning that is DefaultIgnore in one language<br>
> but not in another, this patch<br>
> creates a dummy C only warning in the same group as the existing one<br>
> to get the desired effect.<br>
><br>
> Chandler, is the include of clang/Sema/SemaDiagnostic.h OK in here or<br>
> some extra refactoring is needed?<br>
><br>
> Cheers,<br>
> Rafael<br>
</div></div></blockquote></div><br></div>