[PATCH] D20389: NVPTX: Add supported CL features

Yaxun Liu via llvm-commits llvm-commits at lists.llvm.org
Thu May 19 07:16:09 PDT 2016


yaxunl added a comment.

I have a similar, but yours is more complete. I probably will drop my patch once yours goes through.


================
Comment at: test/Misc/nvptx.languageOptsOpenCL.cl:2
@@ +1,3 @@
+// REQUIRES: nvptx-registered-target
+// RUN: %clang_cc1 -x cl %s -verify -triple nvptx-unknown-unknown -Werror
+// RUN: %clang_cc1 -x cl %s -verify -triple nvptx64-unknown-unknown -Werror
----------------
-Werror is unnecessary, same as below

================
Comment at: test/Misc/nvptx.languageOptsOpenCL.cl:6
@@ +5,3 @@
+
+#pragma EXTENSION cl_clang_storage_class_specifiers: enabled
+#pragma EXTENSION cl_khr_gl_event: enabled
----------------
should be like
  #pragma OPENCL EXTENSION cl_xxx : enable
same for below


Repository:
  rL LLVM

http://reviews.llvm.org/D20389





More information about the llvm-commits mailing list