Skip to content

Commit 76d2304

Browse files
committed
[AMDGPU][Verifier] Check address space of alloca instruction
1 parent 0014b49 commit 76d2304

File tree

2 files changed

+42
-0
lines changed

2 files changed

+42
-0
lines changed

llvm/lib/IR/Verifier.cpp

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4392,6 +4392,12 @@ void Verifier::visitAllocaInst(AllocaInst &AI) {
43924392
verifySwiftErrorValue(&AI);
43934393
}
43944394

4395+
if (TT.isAMDGPU()) {
4396+
Check(AI.getAddressSpace() == DL.getAllocaAddrSpace() ||
4397+
AI.getAddressSpace() == 0,
4398+
"alloca on amdgpu must be in addrspace(0) or addrspace(5)", &AI);
4399+
}
4400+
43954401
visitInstruction(AI);
43964402
}
43974403

llvm/test/Verifier/AMDGPU/alloca.ll

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,36 @@
1+
; RUN: not llvm-as %s --disable-output 2>&1 | FileCheck %s
2+
3+
target triple = "amdgcn-amd-amdhsa"
4+
5+
target datalayout = "A5"
6+
7+
; CHECK: alloca on amdgpu must be in addrspace(0) or addrspace(5)
8+
; CHECK-NEXT: %alloca.1 = alloca i32, align 4, addrspace(1)
9+
; CHECK-NEXT: alloca on amdgpu must be in addrspace(0) or addrspace(5)
10+
; CHECK-NEXT: %alloca.2 = alloca i32, align 4, addrspace(2)
11+
; CHECK-NEXT: alloca on amdgpu must be in addrspace(0) or addrspace(5)
12+
; CHECK-NEXT: %alloca.3 = alloca i32, align 4, addrspace(3)
13+
; CHECK-NEXT: alloca on amdgpu must be in addrspace(0) or addrspace(5)
14+
; CHECK-NEXT: %alloca.4 = alloca i32, align 4, addrspace(4)
15+
; CHECK-NEXT: alloca on amdgpu must be in addrspace(0) or addrspace(5)
16+
; CHECK-NEXT: %alloca.6 = alloca i32, align 4, addrspace(6)
17+
; CHECK-NEXT: alloca on amdgpu must be in addrspace(0) or addrspace(5)
18+
; CHECK-NEXT: %alloca.7 = alloca i32, align 4, addrspace(7)
19+
; CHECK-NEXT: alloca on amdgpu must be in addrspace(0) or addrspace(5)
20+
; CHECK-NEXT: %alloca.8 = alloca i32, align 4, addrspace(8)
21+
; CHECK-NEXT: alloca on amdgpu must be in addrspace(0) or addrspace(5)
22+
; CHECK-NEXT: %alloca.9 = alloca i32, align 4, addrspace(9)
23+
define void @foo() {
24+
entry:
25+
%alloca.0 = alloca i32, align 4
26+
%alloca.1 = alloca i32, align 4, addrspace(1)
27+
%alloca.2 = alloca i32, align 4, addrspace(2)
28+
%alloca.3 = alloca i32, align 4, addrspace(3)
29+
%alloca.4 = alloca i32, align 4, addrspace(4)
30+
%alloca.5 = alloca i32, align 4, addrspace(5)
31+
%alloca.6 = alloca i32, align 4, addrspace(6)
32+
%alloca.7 = alloca i32, align 4, addrspace(7)
33+
%alloca.8 = alloca i32, align 4, addrspace(8)
34+
%alloca.9 = alloca i32, align 4, addrspace(9)
35+
ret void
36+
}

0 commit comments

Comments
 (0)