[clang] [clang][SPIR-V] Addrspace of opencl_global should always be 1 (PR #136753)
Nick Sarnie via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 22 13:01:34 PDT 2025
https://github.com/sarnex updated https://github.com/llvm/llvm-project/pull/136753
>From e78694e52dfdcb6e2b085173952064a806d1d1a8 Mon Sep 17 00:00:00 2001
From: "Sarnie, Nick" <nick.sarnie at intel.com>
Date: Tue, 22 Apr 2025 12:57:36 -0700
Subject: [PATCH 1/2] [clang][SPIR-V] Addrspace of opencl_global should always
be 1
Signed-off-by: Sarnie, Nick <nick.sarnie at intel.com>
---
clang/lib/Basic/Targets/SPIR.h | 4 ++--
clang/test/CodeGenCUDASPIRV/printf.cu | 11 +++++++++++
2 files changed, 13 insertions(+), 2 deletions(-)
create mode 100644 clang/test/CodeGenCUDASPIRV/printf.cu
diff --git a/clang/lib/Basic/Targets/SPIR.h b/clang/lib/Basic/Targets/SPIR.h
index 4509748589b76..577da6fd93d13 100644
--- a/clang/lib/Basic/Targets/SPIR.h
+++ b/clang/lib/Basic/Targets/SPIR.h
@@ -58,8 +58,8 @@ static const unsigned SPIRDefIsPrivMap[] = {
// Used by both the SPIR and SPIR-V targets.
static const unsigned SPIRDefIsGenMap[] = {
4, // Default
- // OpenCL address space values for this map are dummy and they can't be used
- 0, // opencl_global
+ // Some OpenCL address space values for this map are dummy and they can't be used
+ 1, // opencl_global
0, // opencl_local
0, // opencl_constant
0, // opencl_private
diff --git a/clang/test/CodeGenCUDASPIRV/printf.cu b/clang/test/CodeGenCUDASPIRV/printf.cu
new file mode 100644
index 0000000000000..936e920f4a755
--- /dev/null
+++ b/clang/test/CodeGenCUDASPIRV/printf.cu
@@ -0,0 +1,11 @@
+// RUN: %clang_cc1 -fcuda-is-device -triple spirv32 -o - -emit-llvm -x cuda %s | FileCheck --check-prefix=CHECK-SPIRV32 %s
+// RUN: %clang_cc1 -fcuda-is-device -triple spirv64 -o - -emit-llvm -x cuda %s | FileCheck --check-prefix=CHECK-SPIRV64 %s
+
+// CHECK-SPIRV32: @.str = private unnamed_addr addrspace(4) constant [13 x i8] c"Hello World\0A\00", align 1
+// CHECK-SPIRV64: @.str = private unnamed_addr addrspace(1) constant [13 x i8] c"Hello World\0A\00", align 1
+
+extern "C" __attribute__((device)) int printf(const char* format, ...);
+
+__attribute__((global)) void printf_kernel() {
+ printf("Hello World\n");
+}
>From 68a8341a086ade83201a3da08d9a04dc8b424a1c Mon Sep 17 00:00:00 2001
From: "Sarnie, Nick" <nick.sarnie at intel.com>
Date: Tue, 22 Apr 2025 13:01:16 -0700
Subject: [PATCH 2/2] format
Signed-off-by: Sarnie, Nick <nick.sarnie at intel.com>
---
clang/lib/Basic/Targets/SPIR.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/clang/lib/Basic/Targets/SPIR.h b/clang/lib/Basic/Targets/SPIR.h
index 577da6fd93d13..310ef9f2df2c6 100644
--- a/clang/lib/Basic/Targets/SPIR.h
+++ b/clang/lib/Basic/Targets/SPIR.h
@@ -58,7 +58,8 @@ static const unsigned SPIRDefIsPrivMap[] = {
// Used by both the SPIR and SPIR-V targets.
static const unsigned SPIRDefIsGenMap[] = {
4, // Default
- // Some OpenCL address space values for this map are dummy and they can't be used
+ // Some OpenCL address space values for this map are dummy and they can't be
+ // used
1, // opencl_global
0, // opencl_local
0, // opencl_constant
More information about the cfe-commits
mailing list