[PATCH] D20388: AMDGPU: Fix supported CL features

Matt Arsenault via cfe-commits cfe-commits at lists.llvm.org
Tue May 31 10:20:50 PDT 2016


arsenm added inline comments.

================
Comment at: lib/Basic/Targets.cpp:2024-2025
@@ -2023,2 +2023,4 @@
       Builder.defineMacro("__HAS_LDEXPF__");
+    if (hasFP64)
+      Builder.defineMacro("__HAS_FP64__");
   }
----------------
I don't think we need this. I want device macros for other tuning and intrinsic availability reasons. Right now there are builtins that only work on some subtargets but no way to test for that


Repository:
  rL LLVM

http://reviews.llvm.org/D20388





More information about the cfe-commits mailing list