[PATCH] D26476: [AMDGPU] Add support for f16 builtin functions for VI+

[Konstantin Zhuravlyov via cfe-commits]

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