<div dir="ltr"><div>Thanks for this patch! :) Just a few small nits:</div><div><div>- Is there a reason that test/SemaCXX/warn-double-promotion.cpp can't be at test/Sema/warn-double-promotion.c instead?</div></div><div>- Can we make the warning message a bit more specific? Maybe something like "implicit conversion increases floating-point precision: %0 to %1"?</div><div>- Please get rid of the empty line after `else if (TargetBT->getKind() > SourceBT->getKind()) {`</div><div><br></div><div>After those are fixed, LGTM -- will you need me to commit this for you?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 27, 2015 at 9:45 PM, Carl Norum via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi folks,<br>
<br>
I had need of this warning, so I cobbled something together. Can someone please check it out and let me know if it's reasonable? I'm certainly no expert at this stuff. Patch attached for review, thanks!<br>
<span class="HOEnZb"><font color="#888888"><br>
-- Carl<br>
<br>
</font></span><br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>