[PATCH] D17221: Don't combine fp_round (fp_round x) if f80 to f16 is generated

Stephen Hines via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 12 17:18:39 PST 2016


srhines added inline comments.

================
Comment at: test/CodeGen/X86/half.ll:270
@@ +269,3 @@
+; fp_round and the subsequent fptrunc from float to half.
+define half @test_f80trunc_nodagcombine() #0 {
+; CHECK-LABEL: test_f80trunc_nodagcombine:
----------------
ab wrote:
> What do you think of explicitly testing f80->f16 instead?
Pirama, I think you missed this question. :)


Repository:
  rL LLVM

http://reviews.llvm.org/D17221





More information about the llvm-commits mailing list