[PATCH] D26475: [AMDGPU] Update llvm.amdgcn.frexp.exp intrinsic and lower it to v_frexp_exp_i16_f16 instruction

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 9 16:15:42 PST 2016


arsenm added inline comments.


================
Comment at: include/llvm/IR/IntrinsicsAMDGPU.td:181
 def int_amdgcn_frexp_exp : Intrinsic<
-  [llvm_i32_ty], [llvm_anyfloat_ty], [IntrNoMem]
+  [llvm_anyint_ty], [llvm_anyfloat_ty], [IntrNoMem]
 >;
----------------
Could we add a LLVMMatchType to get an int the same width as the FP type to avoid a second mangled parameter? Also could we just leave it as i32? I think it just zeroes the high 16-bits anyway


https://reviews.llvm.org/D26475





More information about the llvm-commits mailing list