[PATCH] [X86][FastISel] Teach how to select float-half conversion intrinsics.

Quentin Colombet qcolombet at apple.com
Tue Feb 17 13:02:56 PST 2015


================
Comment at: test/CodeGen/X86/fast-isel-float-half-convertion.ll:7
@@ +6,3 @@
+entry:
+  %0 = call i16 @llvm.convert.to.fp16.f32(float %a)
+  ret i16 %0
----------------
andreadb wrote:
> qcolombet wrote:
> > Could you add tests with doubles?
> > 
> > I may be wrong but I thought the intrinsic allows any floating type.
> Right, the intrinsic allows any floating point type.
> What if I add those tests into a separate test file maybe an XFAIL test)?
> 
> My concern is that if I add extra tests for doubles in this same file, then the test will start failing because of flag -fast-isel-abort. What do you think? 
Good point.

Sounds good to me.

http://reviews.llvm.org/D7673

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






More information about the llvm-commits mailing list