<div dir="ltr"><div class="gmail_extra">Hi,</div><div class="gmail_extra"><br><div class="gmail_quote">2017-10-25 23:18 GMT+07:00 Aaron Ballman via cfe-dev <span dir="ltr"><<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">On Wed, Oct 25, 2017 at 12:16 PM, Nico Weber via cfe-dev<br>
<<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a>> wrote:<br>
> The post-commit review thread for this warning concluded that this warning<br>
> probably shouldn't be on by default, but looks like it didn't get disabled.<br>
<br></span></blockquote><div> </div><div>This warning is off by default, but in this case it was enabled by command line option.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">
</span>We don't usually add off-by-default diagnostics because almost no one<br>
ever enables them, so perhaps the diagnostic should simply be removed<br>
if it's low-value?<br>
<br></blockquote><div><br></div><div>The two warnings added in r266719 were intended to assist problem solving in the case of</div><div>module builds and they were really helpful.</div><div><br></div><div>There are many warnings disabled by default. For instance, '-Wfloat-conversion', which is</div><div>useful in some cases.</div><div><br></div><div><div><div class="gmail_signature">Thanks,<br>--Serge<br></div></div></div><div><br></div></div></div></div>