Anastasia added inline comments.

================
Comment at: lib/Frontend/InitPreprocessor.cpp:421
@@ +420,3 @@
+    switch (LangOpts.OpenCLVersion) {
+    case 100:
+      Builder.defineMacro("__OPENCL_C_VERSION__", "100");
----------------
Yes, perfectly makes sense. I don't think passing C std flag should be allowed 
since OpenCL is based strictly on C99 and use of other versions are not 
possible anyways.

Nice cleanup!

================
Comment at: lib/Frontend/InitPreprocessor.cpp:439
@@ +438,3 @@
+    Builder.defineMacro("CL_VERSION_1_2", "120");
+    Builder.defineMacro("CL_VERSION_2_0", "200");
+
----------------
Could you update the test please. 

================
Comment at: test/Frontend/stdlang.c:10
@@ -3,1 +9,3 @@
+#ifndef OPENCL_C99
 // expected-no-diagnostics
+#endif
----------------
I think expected-no-diagnostics is only used if -verify is passed.


http://reviews.llvm.org/D19071



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

Reply via email to