[PATCH] D28793: [NVPTX] Auto-upgrade some NVPTX intrinsics to LLVM target-generic code.

Artem Belevich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 20 14:11:28 PST 2017


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





More information about the llvm-commits mailing list