diff options
author | Yaxun Liu <Yaxun.Liu@amd.com> | 2017-10-13 03:37:48 +0000 |
---|---|---|
committer | Yaxun Liu <Yaxun.Liu@amd.com> | 2017-10-13 03:37:48 +0000 |
commit | a86f30ef47b61fe47b424119fbc6f88420310126 (patch) | |
tree | 22440246731ed2fd87490f15d842bece6d6469c2 /test/CodeGenOpenCL | |
parent | 290556e7d3c9fbb74087dc447752b55dbd10ffeb (diff) |
[OpenCL] Add LangAS::opencl_private to represent private address space in AST
Currently Clang uses default address space (0) to represent private address space for OpenCL
in AST. There are two issues with this:
Multiple address spaces including private address space cannot be diagnosed.
There is no mangling for default address space. For example, if private int* is emitted as
i32 addrspace(5)* in IR. It is supposed to be mangled as PUAS5i but it is mangled as
Pi instead.
This patch attempts to represent OpenCL private address space explicitly in AST. It adds
a new enum LangAS::opencl_private and adds it to the variable types which are implicitly
private:
automatic variables without address space qualifier
function parameter
pointee type without address space qualifier (OpenCL 1.2 and below)
Differential Revision: https://reviews.llvm.org/D35082
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@315668 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test/CodeGenOpenCL')
-rw-r--r-- | test/CodeGenOpenCL/address-spaces-mangling.cl | 22 | ||||
-rw-r--r-- | test/CodeGenOpenCL/address-spaces.cl | 67 |
2 files changed, 84 insertions, 5 deletions
diff --git a/test/CodeGenOpenCL/address-spaces-mangling.cl b/test/CodeGenOpenCL/address-spaces-mangling.cl index 3c74c718c2..b6e6b87d9e 100644 --- a/test/CodeGenOpenCL/address-spaces-mangling.cl +++ b/test/CodeGenOpenCL/address-spaces-mangling.cl @@ -1,5 +1,7 @@ -// RUN: %clang_cc1 %s -ffake-address-space-map -faddress-space-map-mangling=yes -triple %itanium_abi_triple -emit-llvm -o - | FileCheck -check-prefix=ASMANG %s -// RUN: %clang_cc1 %s -ffake-address-space-map -faddress-space-map-mangling=no -triple %itanium_abi_triple -emit-llvm -o - | FileCheck -check-prefix=NOASMANG %s +// RUN: %clang_cc1 %s -ffake-address-space-map -faddress-space-map-mangling=yes -triple %itanium_abi_triple -emit-llvm -o - | FileCheck -check-prefixes=ASMANG,ASMAN10 %s +// RUN: %clang_cc1 %s -cl-std=CL2.0 -ffake-address-space-map -faddress-space-map-mangling=yes -triple %itanium_abi_triple -emit-llvm -o - | FileCheck -check-prefixes=ASMANG,ASMAN20 %s +// RUN: %clang_cc1 %s -ffake-address-space-map -faddress-space-map-mangling=no -triple %itanium_abi_triple -emit-llvm -o - | FileCheck -check-prefixes=NOASMANG,NOASMAN10 %s +// RUN: %clang_cc1 %s -cl-std=CL2.0 -ffake-address-space-map -faddress-space-map-mangling=no -triple %itanium_abi_triple -emit-llvm -o - | FileCheck -check-prefixes=NOASMANG,NOASMAN20 %s // We check that the address spaces are mangled the same in both version of OpenCL // RUN: %clang_cc1 %s -triple spir-unknown-unknown -cl-std=CL2.0 -emit-llvm -o - | FileCheck -check-prefix=OCL-20 %s @@ -10,15 +12,17 @@ // warnings, but we do want it for comparison purposes. __attribute__((overloadable)) void ff(int *arg) { } -// ASMANG: @_Z2ffPi -// NOASMANG: @_Z2ffPi +// ASMANG10: @_Z2ffPi +// ASMANG20: @_Z2ffPU3AS4i +// NOASMANG10: @_Z2ffPi +// NOASMANG20: @_Z2ffPU9CLgenerici // OCL-20-DAG: @_Z2ffPU3AS4i // OCL-12-DAG: @_Z2ffPi __attribute__((overloadable)) void f(private int *arg) { } // ASMANG: @_Z1fPi -// NOASMANG: @_Z1fPi +// NOASMANG: @_Z1fPU9CLprivatei // OCL-20-DAG: @_Z1fPi // OCL-12-DAG: @_Z1fPi @@ -42,3 +46,11 @@ void f(constant int *arg) { } // NOASMANG: @_Z1fPU10CLconstanti // OCL-20-DAG: @_Z1fPU3AS2i // OCL-12-DAG: @_Z1fPU3AS2i + +#if __OPENCL_C_VERSION__ >= 200 +__attribute__((overloadable)) +void f(generic int *arg) { } +// ASMANG20: @_Z1fPU3AS4i +// NOASMANG20: @_Z1fPU9CLgenerici +// OCL-20-DAG: @_Z1fPU3AS4i +#endif diff --git a/test/CodeGenOpenCL/address-spaces.cl b/test/CodeGenOpenCL/address-spaces.cl index 488b8f9d48..f78fcf6f8c 100644 --- a/test/CodeGenOpenCL/address-spaces.cl +++ b/test/CodeGenOpenCL/address-spaces.cl @@ -7,6 +7,24 @@ // RUN: %clang_cc1 %s -O0 -triple amdgcn-mesa-mesa3d -emit-llvm -o - | FileCheck --check-prefixes=CHECK,SPIR %s // RUN: %clang_cc1 %s -O0 -triple r600-- -emit-llvm -o - | FileCheck --check-prefixes=CHECK,SPIR %s +// SPIR: %struct.S = type { i32, i32, i32* } +// CL20SPIR: %struct.S = type { i32, i32, i32 addrspace(4)* } +struct S { + int x; + int y; + int *z; +}; + +// CL20-DAG: @g_extern_var = external addrspace(1) global float +// CL20-DAG: @l_extern_var = external addrspace(1) global float +// CL20-DAG: @test_static.l_static_var = internal addrspace(1) global float 0.000000e+00 +// CL20-DAG: @g_static_var = internal addrspace(1) global float 0.000000e+00 + +#ifdef CL20 +// CL20-DAG: @g_s = common addrspace(1) global %struct.S zeroinitializer +struct S g_s; +#endif + // SPIR: i32* %arg // GIZ: i32 addrspace(5)* %arg void f__p(__private int *arg) {} @@ -58,3 +76,52 @@ void f(int *arg) { // CL20-DAG: @f.ii = internal addrspace(1) global i32 0 #endif } + +typedef int int_td; +typedef int *intp_td; +// SPIR: define void @test_typedef(i32 addrspace(1)* %x, i32 addrspace(2)* %y, i32* %z) +void test_typedef(global int_td *x, constant int_td *y, intp_td z) { + *x = *y; + *z = 0; +} + +// SPIR: define void @test_struct() +void test_struct() { + // SPIR: %ps = alloca %struct.S* + // CL20SPIR: %ps = alloca %struct.S addrspace(4)* + struct S *ps; + // SPIR: store i32 0, i32* %x + // CL20SPIR: store i32 0, i32 addrspace(4)* %x + ps->x = 0; +#ifdef CL20 + // CL20SPIR: store i32 0, i32 addrspace(1)* getelementptr inbounds (%struct.S, %struct.S addrspace(1)* @g_s, i32 0, i32 0) + g_s.x = 0; +#endif +} + +// SPIR-LABEL: define void @test_void_par() +void test_void_par(void) {} + +// SPIR-LABEL: define i32 @test_func_return_type() +int test_func_return_type(void) { + return 0; +} + +#ifdef CL20 +extern float g_extern_var; + +// CL20-LABEL: define {{.*}}void @test_extern( +kernel void test_extern(global float *buf) { + extern float l_extern_var; + buf[0] += g_extern_var + l_extern_var; +} + +static float g_static_var; + +// CL20-LABEL: define {{.*}}void @test_static( +kernel void test_static(global float *buf) { + static float l_static_var; + buf[0] += g_static_var + l_static_var; +} + +#endif |