RPCS3/llvm
1
0
Fork 0
mirror of https://github.com/RPCS3/llvm.git synced 2024-12-16 16:37:42 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

AMDGPU/SI: Remove assert from AMDGPUOpenCLImageTypeLowering pass

Browse Source 
Summary:
Instead of asserting when the kernel metadata is different than we expect,
we should just skip lowering that function.  This fixes assertion
failures with OpenCL argument metadata from older LLVM releases.

Reviewers: arsenm

Subscribers: arsenm, llvm-commits

Differential Revision: http://reviews.llvm.org/D13356

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@249073 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Tom Stellard 2015-10-01 21:16:05 +00:00
parent 0955fc3162
commit 5246a3643b
2 changed files with 30 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
lib/Target/AMDGPU/AMDGPUOpenCLImageTypeLoweringPass.cpp
View File

@ -94,8 +94,12 @@ GetFunctionFromMDNode(MDNode *Node) {
return nullptr;
if (!ArgNode->getOperand(0))
return nullptr;
assert(cast<MDString>(ArgNode->getOperand(0))->getString() ==
KernelArgMDNodeNames[i] && "Wrong kernel arg metadata name");
// FIXME: It should be possible to do image lowering when some metadata
// args missing or not in the expected order.
MDString *StringNode = dyn_cast<MDString>(ArgNode->getOperand(0));
if (!StringNode || StringNode->getString() != KernelArgMDNodeNames[i])
return nullptr;
}
return F;

24
test/CodeGen/AMDGPU/opencl-image-metadata.ll Normal file
View File

@ -0,0 +1,24 @@
; RUN: llc < %s -march=r600 -mcpu=redwood | FileCheck --check-prefix=EG --check-prefix=FUNC %s
; RUN: llc < %s -march=amdgcn -mcpu=SI -verify-machineinstrs | FileCheck --check-prefix=SI --check-prefix=FUNC %s
; Make sure the OpenCL Image lowering pass doesn't crash when argument metadata
; is not in expected order.
; EG: CF_END
; SI: s_endpgm
define void @kernel(i32 addrspace(1)* %out) {
entry:
store i32 0, i32 addrspace(1)* %out
ret void
}
attributes #3 = { nounwind }
!opencl.kernels = !{!0}
!0 = !{void (i32 addrspace(1)*)* @kernel, !1, !2, !3, !4, !5}
!1 = !{!"kernel_arg_addr_space", i32 0}
!2 = !{!"kernel_arg_access_qual", !"none"}
!3 = !{!"kernel_arg_type", !"int*"}
!4 = !{!"kernel_arg_type_qual", !""}
!5 = !{!"kernel_arg_name", !""}