[PATCH] [X86] Teach how to custom lower double-to-half conversions under fast-math.

Ahmed Bougacha ahmed.bougacha at gmail.com
Mon Feb 23 09:37:44 PST 2015


Should this be in the target-independent legalizer instead, like FP16_TO_FP?  Only with UnsafeFPMath, of course (I'm a bit uncomfortable with that, but I don't find it shocking, and I see there's precedent.)

Otherwise, the change seems reasonable, thanks!

-Ahmed


================
Comment at: test/CodeGen/X86/fastmath-float-half-conversion.ll:17
@@ +16,3 @@
+; CHECK-NOT: callq  {{_+}}truncxfhf2
+; CHECK: vcvtps2ph $0, %xmm0, %xmm0
+; CHECK: ret
----------------
Not very important, but add a check for fp80->fp32, perhaps?

http://reviews.llvm.org/D7832

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list