From 52a96491e1e4e0d033e39fad87f49ccd871df41d Mon Sep 17 00:00:00 2001 From: Nick Sarnie Date: Thu, 24 Apr 2025 10:20:13 -0400 Subject: [PATCH] [clang][SPIR-V] Addrspace of opencl_global should always be 1 (#136753) This fixes a CUDA SPIR-V regression introduced in https://github.com/llvm/llvm-project/pull/134399. --------- Signed-off-by: Sarnie, Nick --- clang/lib/Basic/Targets/SPIR.h | 5 +++-- clang/test/CodeGenCUDASPIRV/printf.cu | 11 +++++++++++ 2 files changed, 14 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 4509748589b7..310ef9f2df2c 100644 --- a/clang/lib/Basic/Targets/SPIR.h +++ b/clang/lib/Basic/Targets/SPIR.h @@ -58,8 +58,9 @@ 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 000000000000..936e920f4a75 --- /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"); +}