llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang-codegen
@llvm/pr-subscribers-clang
Author: Sven van Haastregt (svenvh)
<details>
<summary>Changes</summary>
Commit 84ee629bc515 ("clang: Remove some pointer bitcasts (#<!-- -->112324)",
2024-10-15) triggered some "Call parameter type does not match function
signature!" errors when using the OpenCL pipe builtin functions under the spir
triple, due to a missing addrspacecast.
This would have been caught by the pipe_builtin.cl test if that had used the
`spir-unknown-unknown` triple, so upgrade the test to use that triple.
---
Full diff: https://github.com/llvm/llvm-project/pull/112514.diff
2 Files Affected:
- (modified) clang/lib/CodeGen/CGBuiltin.cpp (+2-1)
- (modified) clang/test/CodeGenOpenCL/pipe_builtin.cl (+21-23)
``````````diff
diff --git a/clang/lib/CodeGen/CGBuiltin.cpp b/clang/lib/CodeGen/CGBuiltin.cpp
index 12f99d9f1178a9..f6d7db2c204c12 100644
--- a/clang/lib/CodeGen/CGBuiltin.cpp
+++ b/clang/lib/CodeGen/CGBuiltin.cpp
@@ -5657,13 +5657,14 @@ RValue CodeGenFunction::EmitBuiltinExpr(const
GlobalDecl GD, unsigned BuiltinID,
*Arg3 = EmitScalarExpr(E->getArg(3));
llvm::FunctionType *FTy = llvm::FunctionType::get(
Int32Ty, llvm::ArrayRef<llvm::Type *>(ArgTys), false);
+ Value *ACast = Builder.CreateAddrSpaceCast(Arg3, I8PTy);
// We know the third argument is an integer type, but we may need to cast
// it to i32.
if (Arg2->getType() != Int32Ty)
Arg2 = Builder.CreateZExtOrTrunc(Arg2, Int32Ty);
return RValue::get(
EmitRuntimeCall(CGM.CreateRuntimeFunction(FTy, Name),
- {Arg0, Arg1, Arg2, Arg3, PacketSize, PacketAlign}));
+ {Arg0, Arg1, Arg2, ACast, PacketSize, PacketAlign}));
}
}
// OpenCL v2.0 s6.13.16 ,s9.17.3.5 - Built-in pipe reserve read and write
diff --git a/clang/test/CodeGenOpenCL/pipe_builtin.cl
b/clang/test/CodeGenOpenCL/pipe_builtin.cl
index c59f63bab6a458..4e39661f3a3e12 100644
--- a/clang/test/CodeGenOpenCL/pipe_builtin.cl
+++ b/clang/test/CodeGenOpenCL/pipe_builtin.cl
@@ -1,69 +1,67 @@
-// RUN: %clang_cc1 -triple %itanium_abi_triple -emit-llvm
-cl-ext=+cl_khr_subgroups -O0 -cl-std=clc++ -o - %s | FileCheck %s
-// FIXME: Add MS ABI manglings of OpenCL things and remove %itanium_abi_triple
-// above to support OpenCL in the MS C++ ABI.
+// RUN: %clang_cc1 -triple spir-unknown-unknown -emit-llvm
-cl-ext=+cl_khr_subgroups -O0 -cl-std=clc++ -o - %s | FileCheck %s
#pragma OPENCL EXTENSION cl_khr_subgroups : enable
void test1(read_only pipe int p, global int *ptr) {
- // CHECK: call i32 @__read_pipe_2(ptr %{{.*}}, ptr %{{.*}}, i32 4, i32 4)
+ // CHECK: call spir_func i32 @__read_pipe_2(target("spirv.Pipe", 0) %{{.*}},
ptr addrspace(4) %{{.*}}, i32 4, i32 4)
read_pipe(p, ptr);
- // CHECK: call ptr @__reserve_read_pipe(ptr %{{.*}}, i32 {{.*}}, i32 4, i32
4)
+ // CHECK: call spir_func target("spirv.ReserveId")
@__reserve_read_pipe(target("spirv.Pipe", 0) %{{.*}}, i32 {{.*}}, i32 4, i32 4)
reserve_id_t rid = reserve_read_pipe(p, 2);
- // CHECK: call i32 @__read_pipe_4(ptr %{{.*}}, ptr %{{.*}}, i32 {{.*}}, ptr
%{{.*}}, i32 4, i32 4)
+ // CHECK: call spir_func i32 @__read_pipe_4(target("spirv.Pipe", 0) %{{.*}},
ptr addrspace(4) %{{.*}}, i32 4, i32 4)
read_pipe(p, rid, 2, ptr);
- // CHECK: call void @__commit_read_pipe(ptr %{{.*}}, ptr %{{.*}}, i32 4, i32
4)
+ // CHECK: call spir_func void @__commit_read_pipe(target("spirv.Pipe", 0)
%{{.*}}, target("spirv.ReserveId") %{{.*}}, i32 4, i32 4)
commit_read_pipe(p, rid);
}
void test2(write_only pipe int p, global int *ptr) {
- // CHECK: call i32 @__write_pipe_2(ptr %{{.*}}, ptr %{{.*}}, i32 4, i32 4)
+ // CHECK: call spir_func i32 @__write_pipe_2(target("spirv.Pipe", 1)
%{{.*}}, ptr addrspace(4) %{{.*}}, i32 4, i32 4)
write_pipe(p, ptr);
- // CHECK: call ptr @__reserve_write_pipe(ptr %{{.*}}, i32 {{.*}}, i32 4, i32
4)
+ // CHECK: call spir_func target("spirv.ReserveId")
@__reserve_write_pipe(target("spirv.Pipe", 1) %{{.*}}, i32 {{.*}}, i32 4, i32 4)
reserve_id_t rid = reserve_write_pipe(p, 2);
- // CHECK: call i32 @__write_pipe_4(ptr %{{.*}}, ptr %{{.*}}, i32 {{.*}}, ptr
%{{.*}}, i32 4, i32 4)
+ // CHECK: call spir_func i32 @__write_pipe_4(target("spirv.Pipe", 1)
%{{.*}}, ptr addrspace(4) %{{.*}}, i32 4, i32 4)
write_pipe(p, rid, 2, ptr);
- // CHECK: call void @__commit_write_pipe(ptr %{{.*}}, ptr %{{.*}}, i32 4,
i32 4)
+ // CHECK: call spir_func void @__commit_write_pipe(target("spirv.Pipe", 1)
%{{.*}}, target("spirv.ReserveId") %{{.*}}, i32 4, i32 4)
commit_write_pipe(p, rid);
}
void test3(read_only pipe int p, global int *ptr) {
- // CHECK: call ptr @__work_group_reserve_read_pipe(ptr %{{.*}}, i32 {{.*}},
i32 4, i32 4)
+ // CHECK: call spir_func target("spirv.ReserveId")
@__work_group_reserve_read_pipe(target("spirv.Pipe", 0) %{{.*}}, i32 {{.*}},
i32 4, i32 4)
reserve_id_t rid = work_group_reserve_read_pipe(p, 2);
- // CHECK: call void @__work_group_commit_read_pipe(ptr %{{.*}}, ptr %{{.*}},
i32 4, i32 4)
+ // CHECK: call spir_func void
@__work_group_commit_read_pipe(target("spirv.Pipe", 0) %{{.*}},
target("spirv.ReserveId") %{{.*}}, i32 4, i32 4)
work_group_commit_read_pipe(p, rid);
}
void test4(write_only pipe int p, global int *ptr) {
- // CHECK: call ptr @__work_group_reserve_write_pipe(ptr %{{.*}}, i32 {{.*}},
i32 4, i32 4)
+ // CHECK: call spir_func target("spirv.ReserveId")
@__work_group_reserve_write_pipe(target("spirv.Pipe", 1) %{{.*}}, i32 {{.*}},
i32 4, i32 4)
reserve_id_t rid = work_group_reserve_write_pipe(p, 2);
- // CHECK: call void @__work_group_commit_write_pipe(ptr %{{.*}}, ptr
%{{.*}}, i32 4, i32 4)
+ // CHECK: call spir_func void
@__work_group_commit_write_pipe(target("spirv.Pipe", 1) %{{.*}},
target("spirv.ReserveId") %{{.*}}, i32 4, i32 4)
work_group_commit_write_pipe(p, rid);
}
void test5(read_only pipe int p, global int *ptr) {
- // CHECK: call ptr @__sub_group_reserve_read_pipe(ptr %{{.*}}, i32 {{.*}},
i32 4, i32 4)
+ // CHECK: call spir_func target("spirv.ReserveId")
@__sub_group_reserve_read_pipe(target("spirv.Pipe", 0) %{{.*}}, i32 {{.*}}, i32
4, i32 4)
reserve_id_t rid = sub_group_reserve_read_pipe(p, 2);
- // CHECK: call void @__sub_group_commit_read_pipe(ptr %{{.*}}, ptr %{{.*}},
i32 4, i32 4)
+ // CHECK: call spir_func void
@__sub_group_commit_read_pipe(target("spirv.Pipe", 0) %{{.*}},
target("spirv.ReserveId") %{{.*}}, i32 4, i32 4)
sub_group_commit_read_pipe(p, rid);
}
void test6(write_only pipe int p, global int *ptr) {
- // CHECK: call ptr @__sub_group_reserve_write_pipe(ptr %{{.*}}, i32 {{.*}},
i32 4, i32 4)
+ // CHECK: call spir_func target("spirv.ReserveId")
@__sub_group_reserve_write_pipe(target("spirv.Pipe", 1) %{{.*}}, i32 {{.*}},
i32 4, i32 4)
reserve_id_t rid = sub_group_reserve_write_pipe(p, 2);
- // CHECK: call void @__sub_group_commit_write_pipe(ptr %{{.*}}, ptr %{{.*}},
i32 4, i32 4)
+ // CHECK: call spir_func void
@__sub_group_commit_write_pipe(target("spirv.Pipe", 1) %{{.*}},
target("spirv.ReserveId") %{{.*}}, i32 4, i32 4)
sub_group_commit_write_pipe(p, rid);
}
void test7(read_only pipe int p, global int *ptr) {
- // CHECK: call i32 @__get_pipe_num_packets_ro(ptr %{{.*}}, i32 4, i32 4)
+ // CHECK: call spir_func i32 @__get_pipe_num_packets_ro(target("spirv.Pipe",
0) %{{.*}}, i32 4, i32 4)
*ptr = get_pipe_num_packets(p);
- // CHECK: call i32 @__get_pipe_max_packets_ro(ptr %{{.*}}, i32 4, i32 4)
+ // CHECK: call spir_func i32 @__get_pipe_max_packets_ro(target("spirv.Pipe",
0) %{{.*}}, i32 4, i32 4)
*ptr = get_pipe_max_packets(p);
}
void test8(write_only pipe int p, global int *ptr) {
- // CHECK: call i32 @__get_pipe_num_packets_wo(ptr %{{.*}}, i32 4, i32 4)
+ // CHECK: call spir_func i32 @__get_pipe_num_packets_wo(target("spirv.Pipe",
1) %{{.*}}, i32 4, i32 4)
*ptr = get_pipe_num_packets(p);
- // CHECK: call i32 @__get_pipe_max_packets_wo(ptr %{{.*}}, i32 4, i32 4)
+ // CHECK: call spir_func i32 @__get_pipe_max_packets_wo(target("spirv.Pipe",
1) %{{.*}}, i32 4, i32 4)
*ptr = get_pipe_max_packets(p);
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/112514
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
