<div dir="ltr">Committed as r251588 -- thanks again!<div><br></div><div>FYI: I'm assuming that you meant to change the test to warn-double-promotion.c instead of leaving it as warn-double-promotion.cpp, so I made that fix for you prior to committing. If this is isn't something you wanted, please let me know. :)</div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-10-28 17:02 GMT-07:00 Carl Norum <span dir="ltr"><<a href="mailto:carl.norum@apple.com" target="_blank">carl.norum@apple.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">OK, here you go!<br>
<span class="HOEnZb"><font color="#888888"><br>
-- Carl<br>
<br>
<br>
</font></span><br><br>
> On Oct 28, 2015, at 4:53 PM, Carl Norum <<a href="mailto:carl.norum@apple.com">carl.norum@apple.com</a>> wrote:<br>
><br>
><br>
>> On Oct 28, 2015, at 4:37 PM, George Burgess IV <<a href="mailto:george.burgess.iv@gmail.com">george.burgess.iv@gmail.com</a>> wrote:<br>
>><br>
>> Thanks for this patch! :) Just a few small nits:<br>
>> - Is there a reason that test/SemaCXX/warn-double-promotion.cpp can't be at test/Sema/warn-double-promotion.c instead?<br>
><br>
> No reason; I just found warn-float-conversion.c in there and this code is similar. Can move easily.<br>
><br>
>> - Can we make the warning message a bit more specific? Maybe something like "implicit conversion increases floating-point precision: %0 to %1"?<br>
><br>
> Sure thing.<br>
><br>
>> - Please get rid of the empty line after `else if (TargetBT->getKind() > SourceBT->getKind()) {`<br>
><br>
> NP. Will send a patch shortly.<br>
><br>
>> After those are fixed, LGTM -- will you need me to commit this for you?<br>
><br>
> I think so yes.<br>
><br>
> -- Carl<br>
><br>
><br>
>> On Tue, Oct 27, 2015 at 9:45 PM, Carl Norum via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> wrote:<br>
>> 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>
>><br>
>> -- Carl<br>
>><br>
>><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>
>><br>
><br>
<br>
<br></blockquote></div><br></div>