diff options
| author | Yaxun Liu <Yaxun.Liu@amd.com> | 2017-04-17 20:10:44 +0000 |
|---|---|---|
| committer | Yaxun Liu <Yaxun.Liu@amd.com> | 2017-04-17 20:10:44 +0000 |
| commit | d7523283a7198d0e919f8ada0eeadddef3ccbaaa (patch) | |
| tree | d355616dd0c86ad1e2a8b81e6097d8a63a8215d3 /clang | |
| parent | ef700d550e7349c94081c1dc93eac6a5be2fc1f5 (diff) | |
| download | bcm5719-llvm-d7523283a7198d0e919f8ada0eeadddef3ccbaaa.tar.gz bcm5719-llvm-d7523283a7198d0e919f8ada0eeadddef3ccbaaa.zip | |
CodeGen: Let byval parameter use alloca address space
Differential Revision: https://reviews.llvm.org/D32133
llvm-svn: 300487
Diffstat (limited to 'clang')
| -rw-r--r-- | clang/lib/CodeGen/CGCall.cpp | 5 | ||||
| -rw-r--r-- | clang/test/CodeGenOpenCL/byval.cl | 18 |
2 files changed, 21 insertions, 2 deletions
diff --git a/clang/lib/CodeGen/CGCall.cpp b/clang/lib/CodeGen/CGCall.cpp index 8af32055fc4..8e25dc2cd70 100644 --- a/clang/lib/CodeGen/CGCall.cpp +++ b/clang/lib/CodeGen/CGCall.cpp @@ -1586,9 +1586,10 @@ CodeGenTypes::GetFunctionType(const CGFunctionInfo &FI) { case ABIArgInfo::Indirect: { assert(NumIRArgs == 1); - // indirect arguments are always on the stack, which is addr space #0. + // indirect arguments are always on the stack, which is alloca addr space. llvm::Type *LTy = ConvertTypeForMem(it->type); - ArgTypes[FirstIRArg] = LTy->getPointerTo(); + ArgTypes[FirstIRArg] = LTy->getPointerTo( + CGM.getDataLayout().getAllocaAddrSpace()); break; } diff --git a/clang/test/CodeGenOpenCL/byval.cl b/clang/test/CodeGenOpenCL/byval.cl new file mode 100644 index 00000000000..1a8105c5207 --- /dev/null +++ b/clang/test/CodeGenOpenCL/byval.cl @@ -0,0 +1,18 @@ +// RUN: %clang_cc1 -emit-llvm -o - -triple amdgcn %s | FileCheck %s +// RUN: %clang_cc1 -emit-llvm -o - -triple amdgcn---amdgizcl %s | FileCheck %s -check-prefix=AMDGIZ + +struct A { + int x[100]; +}; + +int f(struct A a); + +int g() { + struct A a; + // CHECK: call i32 @f(%struct.A* byval{{.*}}%a) + // AMDGIZ: call i32 @f(%struct.A addrspace(5)* byval{{.*}}%a) + return f(a); +} + +// CHECK: declare i32 @f(%struct.A* byval{{.*}}) +// AMDGIZ: declare i32 @f(%struct.A addrspace(5)* byval{{.*}}) |
