[PATCH] D48573: [AMDGPU] Add llvm.amdgcn.fmad.ftz intrinsic

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: include/llvm/IR/IntrinsicsAMDGPU.td:364
+// v_mad_f32/v_mac_f32, selected regardless of denorm support.
+def int_amdgcn_fmad_ftz :
+  Intrinsic<[llvm_float_ty],
----------------
rampitec wrote:
> arsenm wrote:
> > Does v_mad_f16 flush denormals? I don't think it does
> It does. It cannot preserve denormals. It is even lowered into FMAD_FTZ, which is lowered to mad. The new part is this intrinsic sitting above the existing SDNode.
In that case I would make this intrinsic type mangled and make it work for f16 as well


https://reviews.llvm.org/D48573