[PATCH] D21031: [OpenCL] Allow -cl-std and other standard -cl- options in driver
Anastasia Stulova via cfe-commits
cfe-commits at lists.llvm.org
Tue Jun 21 05:19:16 PDT 2016
Anastasia added inline comments.
================
Comment at: include/clang/Basic/DiagnosticFrontendKinds.td:220
@@ +219,2 @@
+def warn_option_depre_ocl_version : Warning<
+ "Option %0 is deprecated for this OpenCL version">, InGroup<Deprecated>;
----------------
Looks good. If we could output the current OpenCL version as well to match the diagnostic style from http://reviews.llvm.org/D19780, it would be nice.
================
Comment at: test/Driver/opencl.cl:32
@@ -14,2 +31,3 @@
+// CHECK-DENORMS-ARE-ZERO: .*clang.* "-cc1" .* "-cl-denorms-are-zero"
kernel void func(void);
----------------
Could you also test the new diagnostic message you are adding please?
http://reviews.llvm.org/D21031
More information about the cfe-commits
mailing list