[PATCH] D43414: AMDGPU: Define FP_FAST_FMA{F} macros for amdgcn

Brian Sumner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 23 15:11:02 PST 2018


b-sumner added inline comments.


================
Comment at: lib/Basic/Targets/AMDGPU.cpp:347
+  if (GPU.HasFMAF)
     Builder.defineMacro("__HAS_FMAF__");
+  if (GPU.HasFP64)
----------------
I'm not sure why this is here.  No languages we support have this AFAIK.  We should probably add a comment that this is deprecated and remove it in a year or so.


https://reviews.llvm.org/D43414





More information about the llvm-commits mailing list