tra accepted this revision. tra added a comment. This revision is now accepted and ready to land.
There are couple of bits to be deleted. LGTM otherwise. ================ Comment at: llvm/include/llvm/IR/IntrinsicsNVVM.td:671-672 def int_nvvm_h2f : GCCBuiltin<"__nvvm_h2f">, Intrinsic<[llvm_float_ty], [llvm_i16_ty], [IntrNoMem]>; ---------------- This one should be gone, too. ================ Comment at: llvm/lib/Target/NVPTX/NVPTXIntrinsics.td:770-771 def : Pat<(int_nvvm_h2f Int16Regs:$a), (CVT_f32_f16 (BITCONVERT_16_I2F Int16Regs:$a), CvtNONE)>; ---------------- Same here. https://reviews.llvm.org/D28793 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits