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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to