kzhuravl added inline comments.

================
Comment at: test/CodeGenOpenCL/builtins-amdgcn-error-f16-class.cl:1-9
+// REQUIRES: amdgpu-registered-target
+// RUN: %clang_cc1 -triple amdgcn-unknown-amdhsa -target-cpu tahiti -verify -S 
-o - %s
+
+#pragma OPENCL EXTENSION cl_khr_fp16 : enable
+
+void test_class_f16(global half* out, half a, int b)
+{
----------------
arsenm wrote:
> These tests can all be in the same file 
The problem is if I put them into the same file only the first function gives 
the error `'__builtin_amdgcn_*h' needs target feature 16-bit-insts` and I did 
not have time to investigate why, but I was planning to do it once I finish 
something else.

Would leaving them in the separate files be acceptable short-term?


https://reviews.llvm.org/D26476



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

Reply via email to