<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Nov 28, 2013 at 1:20 AM, Nadav Rotem <span dir="ltr"><<a href="mailto:nrotem@apple.com" target="_blank">nrotem@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Steve!<br>
<br>
You are the expert in this domain so I trust you that the code is correct :)<br>
<br>
Can you please add a CHECK: line for the function name and for ‘ret’ in your tests? It helps in debugging failing tests.  Also, InstCombine/fpextend.ll probably needs a CHECK line.<br></blockquote><div><br></div><div>CHECK-LABEL for the function name also gives extra goodness.</div>
<div><br></div><div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
The rest LGTM.<br>
<br>
Thanks,<br>
Nadav<br>
<div><div class="h5"><br>
<br>
On Nov 27, 2013, at 10:48 AM, Stephen Canon <<a href="mailto:scanon@apple.com">scanon@apple.com</a>> wrote:<br>
<br>
> InstCombiner::visitFPTrunc is currently overzealous and does some "optimizations" that it shouldn't (because they change results).  The attached patch reins it in so that we still get the benefit of this optimization for common cases, but we get rid of the misbehavior, and adds some test cases for situations where the optimization *shouldn't* happen.<br>

><br>
> Thanks to Pascal Quoc for bringing the issue to my attention (<a href="http://blog.frama-c.com/index.php?post/2013/09/25/The-problem-with-differential-testing-is-that-at-least-one-of-the-compilers-must-get-it-right" target="_blank">http://blog.frama-c.com/index.php?post/2013/09/25/The-problem-with-differential-testing-is-that-at-least-one-of-the-compilers-must-get-it-right</a>).<br>

><br>
> This is also <rdar://problem/15080092>.<br>
><br>
> Thanks,<br>
> – Steve<br>
><br>
</div></div>> <fpextend_patch.txt>_______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div></div>