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

Reply via email to