https://github.com/shiltian updated https://github.com/llvm/llvm-project/pull/99546
>From de113a6635eab7fc37eea9f8c189b208c9be869a Mon Sep 17 00:00:00 2001 From: Shilei Tian <i...@tianshilei.me> Date: Thu, 18 Jul 2024 14:28:27 -0400 Subject: [PATCH] [Clang][AMDGPU] Use unsigned data type for `__builtin_amdgcn_raw_buffer_store_*` --- clang/include/clang/Basic/BuiltinsAMDGPU.def | 12 +-- .../builtins-amdgcn-raw-buffer-store.cl | 84 +++++++++---------- .../builtins-amdgcn-raw-buffer-store-error.cl | 26 +++--- 3 files changed, 61 insertions(+), 61 deletions(-) diff --git a/clang/include/clang/Basic/BuiltinsAMDGPU.def b/clang/include/clang/Basic/BuiltinsAMDGPU.def index e62315eea277a..d9c1f7da66d78 100644 --- a/clang/include/clang/Basic/BuiltinsAMDGPU.def +++ b/clang/include/clang/Basic/BuiltinsAMDGPU.def @@ -149,12 +149,12 @@ BUILTIN(__builtin_amdgcn_mqsad_pk_u16_u8, "WUiWUiUiWUi", "nc") BUILTIN(__builtin_amdgcn_mqsad_u32_u8, "V4UiWUiUiV4Ui", "nc") BUILTIN(__builtin_amdgcn_make_buffer_rsrc, "Qbv*sii", "nc") -BUILTIN(__builtin_amdgcn_raw_buffer_store_b8, "vcQbiiIi", "n") -BUILTIN(__builtin_amdgcn_raw_buffer_store_b16, "vsQbiiIi", "n") -BUILTIN(__builtin_amdgcn_raw_buffer_store_b32, "viQbiiIi", "n") -BUILTIN(__builtin_amdgcn_raw_buffer_store_b64, "vV2iQbiiIi", "n") -BUILTIN(__builtin_amdgcn_raw_buffer_store_b96, "vV3iQbiiIi", "n") -BUILTIN(__builtin_amdgcn_raw_buffer_store_b128, "vV4iQbiiIi", "n") +BUILTIN(__builtin_amdgcn_raw_buffer_store_b8, "vUcQbiiIi", "n") +BUILTIN(__builtin_amdgcn_raw_buffer_store_b16, "vUsQbiiIi", "n") +BUILTIN(__builtin_amdgcn_raw_buffer_store_b32, "vUiQbiiIi", "n") +BUILTIN(__builtin_amdgcn_raw_buffer_store_b64, "vV2UiQbiiIi", "n") +BUILTIN(__builtin_amdgcn_raw_buffer_store_b96, "vV3UiQbiiIi", "n") +BUILTIN(__builtin_amdgcn_raw_buffer_store_b128, "vV4UiQbiiIi", "n") //===----------------------------------------------------------------------===// // Ballot builtins. diff --git a/clang/test/CodeGenOpenCL/builtins-amdgcn-raw-buffer-store.cl b/clang/test/CodeGenOpenCL/builtins-amdgcn-raw-buffer-store.cl index 37975d59730c5..3621c9451275c 100644 --- a/clang/test/CodeGenOpenCL/builtins-amdgcn-raw-buffer-store.cl +++ b/clang/test/CodeGenOpenCL/builtins-amdgcn-raw-buffer-store.cl @@ -2,171 +2,171 @@ // REQUIRES: amdgpu-registered-target // RUN: %clang_cc1 -triple amdgcn-unknown-unknown -target-cpu verde -emit-llvm -o - %s | FileCheck %s -typedef char i8; -typedef short i16; -typedef int i32; -typedef int i64 __attribute__((ext_vector_type(2))); -typedef int i96 __attribute__((ext_vector_type(3))); -typedef int i128 __attribute__((ext_vector_type(4))); +typedef unsigned char u8; +typedef unsigned short u16; +typedef unsigned int u32; +typedef unsigned int u64 __attribute__((ext_vector_type(2))); +typedef unsigned int u96 __attribute__((ext_vector_type(3))); +typedef unsigned int u128 __attribute__((ext_vector_type(4))); // CHECK-LABEL: @test_amdgcn_raw_ptr_buffer_store_b8( // CHECK-NEXT: entry: -// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.i8(i8 [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], i32 0, i32 0, i32 0) +// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.i8(u8 [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], u32 0, u32 0, u32 0) // CHECK-NEXT: ret void // -void test_amdgcn_raw_ptr_buffer_store_b8(i8 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { +void test_amdgcn_raw_ptr_buffer_store_b8(u8 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { __builtin_amdgcn_raw_buffer_store_b8(vdata, rsrc, /*offset=*/0, /*soffset=*/0, /*aux=*/0); } // CHECK-LABEL: @test_amdgcn_raw_ptr_buffer_store_b16( // CHECK-NEXT: entry: -// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.i16(i16 [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], i32 0, i32 0, i32 0) +// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.i16(u16 [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], u32 0, u32 0, u32 0) // CHECK-NEXT: ret void // -void test_amdgcn_raw_ptr_buffer_store_b16(i16 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { +void test_amdgcn_raw_ptr_buffer_store_b16(u16 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { __builtin_amdgcn_raw_buffer_store_b16(vdata, rsrc, /*offset=*/0, /*soffset=*/0, /*aux=*/0); } // CHECK-LABEL: @test_amdgcn_raw_ptr_buffer_store_b32( // CHECK-NEXT: entry: -// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.i32(i32 [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], i32 0, i32 0, i32 0) +// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.i32(u32 [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], u32 0, u32 0, u32 0) // CHECK-NEXT: ret void // -void test_amdgcn_raw_ptr_buffer_store_b32(i32 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { +void test_amdgcn_raw_ptr_buffer_store_b32(u32 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { __builtin_amdgcn_raw_buffer_store_b32(vdata, rsrc, /*offset=*/0, /*soffset=*/0, /*aux=*/0); } // CHECK-LABEL: @test_amdgcn_raw_ptr_buffer_store_b64( // CHECK-NEXT: entry: -// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.v2i32(<2 x i32> [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], i32 0, i32 0, i32 0) +// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.v2i32(<2 x u32> [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], u32 0, u32 0, u32 0) // CHECK-NEXT: ret void // -void test_amdgcn_raw_ptr_buffer_store_b64(i64 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { +void test_amdgcn_raw_ptr_buffer_store_b64(u64 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { __builtin_amdgcn_raw_buffer_store_b64(vdata, rsrc, /*offset=*/0, /*soffset=*/0, /*aux=*/0); } // CHECK-LABEL: @test_amdgcn_raw_ptr_buffer_store_b96( // CHECK-NEXT: entry: -// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.v3i32(<3 x i32> [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], i32 0, i32 0, i32 0) +// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.v3i32(<3 x u32> [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], u32 0, u32 0, u32 0) // CHECK-NEXT: ret void // -void test_amdgcn_raw_ptr_buffer_store_b96(i96 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { +void test_amdgcn_raw_ptr_buffer_store_b96(u96 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { __builtin_amdgcn_raw_buffer_store_b96(vdata, rsrc, /*offset=*/0, /*soffset=*/0, /*aux=*/0); } // CHECK-LABEL: @test_amdgcn_raw_ptr_buffer_store_b128( // CHECK-NEXT: entry: -// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.v4i32(<4 x i32> [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], i32 0, i32 0, i32 0) +// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.v4i32(<4 x u32> [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], u32 0, u32 0, u32 0) // CHECK-NEXT: ret void // -void test_amdgcn_raw_ptr_buffer_store_b128(i128 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { +void test_amdgcn_raw_ptr_buffer_store_b128(u128 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { __builtin_amdgcn_raw_buffer_store_b128(vdata, rsrc, /*offset=*/0, /*soffset=*/0, /*aux=*/0); } // CHECK-LABEL: @test_amdgcn_raw_ptr_buffer_store_b8_non_const_offset( // CHECK-NEXT: entry: -// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.i8(i8 [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], i32 [[OFFSET:%.*]], i32 0, i32 0) +// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.i8(u8 [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], u32 [[OFFSET:%.*]], u32 0, u32 0) // CHECK-NEXT: ret void // -void test_amdgcn_raw_ptr_buffer_store_b8_non_const_offset(i8 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { +void test_amdgcn_raw_ptr_buffer_store_b8_non_const_offset(u8 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { __builtin_amdgcn_raw_buffer_store_b8(vdata, rsrc, offset, /*soffset=*/0, /*aux=*/0); } // CHECK-LABEL: @test_amdgcn_raw_ptr_buffer_store_b16_non_const_offset( // CHECK-NEXT: entry: -// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.i16(i16 [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], i32 [[OFFSET:%.*]], i32 0, i32 0) +// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.i16(u16 [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], u32 [[OFFSET:%.*]], u32 0, u32 0) // CHECK-NEXT: ret void // -void test_amdgcn_raw_ptr_buffer_store_b16_non_const_offset(i16 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { +void test_amdgcn_raw_ptr_buffer_store_b16_non_const_offset(u16 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { __builtin_amdgcn_raw_buffer_store_b16(vdata, rsrc, offset, /*soffset=*/0, /*aux=*/0); } // CHECK-LABEL: @test_amdgcn_raw_ptr_buffer_store_b32_non_const_offset( // CHECK-NEXT: entry: -// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.i32(i32 [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], i32 [[OFFSET:%.*]], i32 0, i32 0) +// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.i32(u32 [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], u32 [[OFFSET:%.*]], u32 0, u32 0) // CHECK-NEXT: ret void // -void test_amdgcn_raw_ptr_buffer_store_b32_non_const_offset(i32 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { +void test_amdgcn_raw_ptr_buffer_store_b32_non_const_offset(u32 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { __builtin_amdgcn_raw_buffer_store_b32(vdata, rsrc, offset, /*soffset=*/0, /*aux=*/0); } // CHECK-LABEL: @test_amdgcn_raw_ptr_buffer_store_b64_non_const_offset( // CHECK-NEXT: entry: -// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.v2i32(<2 x i32> [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], i32 [[OFFSET:%.*]], i32 0, i32 0) +// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.v2i32(<2 x u32> [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], u32 [[OFFSET:%.*]], u32 0, u32 0) // CHECK-NEXT: ret void // -void test_amdgcn_raw_ptr_buffer_store_b64_non_const_offset(i64 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { +void test_amdgcn_raw_ptr_buffer_store_b64_non_const_offset(u64 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { __builtin_amdgcn_raw_buffer_store_b64(vdata, rsrc, offset, /*soffset=*/0, /*aux=*/0); } // CHECK-LABEL: @test_amdgcn_raw_ptr_buffer_store_b96_non_const_offset( // CHECK-NEXT: entry: -// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.v3i32(<3 x i32> [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], i32 [[OFFSET:%.*]], i32 0, i32 0) +// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.v3i32(<3 x u32> [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], u32 [[OFFSET:%.*]], u32 0, u32 0) // CHECK-NEXT: ret void // -void test_amdgcn_raw_ptr_buffer_store_b96_non_const_offset(i96 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { +void test_amdgcn_raw_ptr_buffer_store_b96_non_const_offset(u96 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { __builtin_amdgcn_raw_buffer_store_b96(vdata, rsrc, offset, /*soffset=*/0, /*aux=*/0); } // CHECK-LABEL: @test_amdgcn_raw_ptr_buffer_store_b128_non_const_offset( // CHECK-NEXT: entry: -// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.v4i32(<4 x i32> [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], i32 [[OFFSET:%.*]], i32 0, i32 0) +// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.v4i32(<4 x u32> [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], u32 [[OFFSET:%.*]], u32 0, u32 0) // CHECK-NEXT: ret void // -void test_amdgcn_raw_ptr_buffer_store_b128_non_const_offset(i128 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { +void test_amdgcn_raw_ptr_buffer_store_b128_non_const_offset(u128 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { __builtin_amdgcn_raw_buffer_store_b128(vdata, rsrc, offset, /*soffset=*/0, /*aux=*/0); } // CHECK-LABEL: @test_amdgcn_raw_ptr_buffer_store_b8_non_const_soffset( // CHECK-NEXT: entry: -// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.i8(i8 [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], i32 0, i32 [[SOFFSET:%.*]], i32 0) +// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.i8(u8 [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], u32 0, u32 [[SOFFSET:%.*]], u32 0) // CHECK-NEXT: ret void // -void test_amdgcn_raw_ptr_buffer_store_b8_non_const_soffset(i8 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { +void test_amdgcn_raw_ptr_buffer_store_b8_non_const_soffset(u8 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { __builtin_amdgcn_raw_buffer_store_b8(vdata, rsrc, /*offset=*/0, soffset, /*aux=*/0); } // CHECK-LABEL: @test_amdgcn_raw_ptr_buffer_store_b16_non_const_soffset( // CHECK-NEXT: entry: -// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.i16(i16 [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], i32 0, i32 [[SOFFSET:%.*]], i32 0) +// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.i16(u16 [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], u32 0, u32 [[SOFFSET:%.*]], u32 0) // CHECK-NEXT: ret void // -void test_amdgcn_raw_ptr_buffer_store_b16_non_const_soffset(i16 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { +void test_amdgcn_raw_ptr_buffer_store_b16_non_const_soffset(u16 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { __builtin_amdgcn_raw_buffer_store_b16(vdata, rsrc, /*offset=*/0, soffset, /*aux=*/0); } // CHECK-LABEL: @test_amdgcn_raw_ptr_buffer_store_b32_non_const_soffset( // CHECK-NEXT: entry: -// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.i32(i32 [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], i32 0, i32 [[SOFFSET:%.*]], i32 0) +// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.i32(u32 [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], u32 0, u32 [[SOFFSET:%.*]], u32 0) // CHECK-NEXT: ret void // -void test_amdgcn_raw_ptr_buffer_store_b32_non_const_soffset(i32 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { +void test_amdgcn_raw_ptr_buffer_store_b32_non_const_soffset(u32 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { __builtin_amdgcn_raw_buffer_store_b32(vdata, rsrc, /*offset=*/0, soffset, /*aux=*/0); } // CHECK-LABEL: @test_amdgcn_raw_ptr_buffer_store_b64_non_const_soffset( // CHECK-NEXT: entry: -// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.v2i32(<2 x i32> [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], i32 0, i32 [[SOFFSET:%.*]], i32 0) +// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.v2i32(<2 x u32> [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], u32 0, u32 [[SOFFSET:%.*]], u32 0) // CHECK-NEXT: ret void // -void test_amdgcn_raw_ptr_buffer_store_b64_non_const_soffset(i64 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { +void test_amdgcn_raw_ptr_buffer_store_b64_non_const_soffset(u64 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { __builtin_amdgcn_raw_buffer_store_b64(vdata, rsrc, /*offset=*/0, soffset, /*aux=*/0); } // CHECK-LABEL: @test_amdgcn_raw_ptr_buffer_store_b96_non_const_soffset( // CHECK-NEXT: entry: -// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.v3i32(<3 x i32> [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], i32 0, i32 [[SOFFSET:%.*]], i32 0) +// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.v3i32(<3 x u32> [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], u32 0, u32 [[SOFFSET:%.*]], u32 0) // CHECK-NEXT: ret void // -void test_amdgcn_raw_ptr_buffer_store_b96_non_const_soffset(i96 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { +void test_amdgcn_raw_ptr_buffer_store_b96_non_const_soffset(u96 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { __builtin_amdgcn_raw_buffer_store_b96(vdata, rsrc, /*offset=*/0, soffset, /*aux=*/0); } // CHECK-LABEL: @test_amdgcn_raw_ptr_buffer_store_b128_non_const_soffset( // CHECK-NEXT: entry: -// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.v4i32(<4 x i32> [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], i32 0, i32 [[SOFFSET:%.*]], i32 0) +// CHECK-NEXT: tail call void @llvm.amdgcn.raw.ptr.buffer.store.v4i32(<4 x u32> [[VDATA:%.*]], ptr addrspace(8) [[RSRC:%.*]], u32 0, u32 [[SOFFSET:%.*]], u32 0) // CHECK-NEXT: ret void // -void test_amdgcn_raw_ptr_buffer_store_b128_non_const_soffset(i128 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { +void test_amdgcn_raw_ptr_buffer_store_b128_non_const_soffset(u128 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset) { __builtin_amdgcn_raw_buffer_store_b128(vdata, rsrc, /*offset=*/0, soffset, /*aux=*/0); } diff --git a/clang/test/SemaOpenCL/builtins-amdgcn-raw-buffer-store-error.cl b/clang/test/SemaOpenCL/builtins-amdgcn-raw-buffer-store-error.cl index 356c031640483..8df8b91ebbf4a 100644 --- a/clang/test/SemaOpenCL/builtins-amdgcn-raw-buffer-store-error.cl +++ b/clang/test/SemaOpenCL/builtins-amdgcn-raw-buffer-store-error.cl @@ -3,33 +3,33 @@ #pragma OPENCL EXTENSION cl_khr_fp16 : enable -typedef char i8; -typedef short i16; -typedef int i32; -typedef int i64 __attribute__((ext_vector_type(2))); -typedef int i96 __attribute__((ext_vector_type(3))); -typedef int i128 __attribute__((ext_vector_type(4))); - -void test_amdgcn_raw_ptr_buffer_store_b8(i8 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset, int aux) { +typedef unsigned char u8; +typedef unsigned short u16; +typedef unsigned int u32; +typedef unsigned int u64 __attribute__((ext_vector_type(2))); +typedef unsigned int u96 __attribute__((ext_vector_type(3))); +typedef unsigned int u128 __attribute__((ext_vector_type(4))); + +void test_amdgcn_raw_ptr_buffer_store_b8(u8 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset, int aux) { __builtin_amdgcn_raw_buffer_store_b8(vdata, rsrc, /*offset=*/0, /*soffset=*/0, aux); //expected-error{{argument to '__builtin_amdgcn_raw_buffer_store_b8' must be a constant integer}} } -void test_amdgcn_raw_ptr_buffer_store_b16(i16 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset, int aux) { +void test_amdgcn_raw_ptr_buffer_store_b16(u16 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset, int aux) { __builtin_amdgcn_raw_buffer_store_b16(vdata, rsrc, /*offset=*/0, /*soffset=*/0, aux); //expected-error{{argument to '__builtin_amdgcn_raw_buffer_store_b16' must be a constant integer}} } -void test_amdgcn_raw_ptr_buffer_store_b32(i32 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset, int aux) { +void test_amdgcn_raw_ptr_buffer_store_b32(u32 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset, int aux) { __builtin_amdgcn_raw_buffer_store_b32(vdata, rsrc, /*offset=*/0, /*soffset=*/0, aux); //expected-error{{argument to '__builtin_amdgcn_raw_buffer_store_b32' must be a constant integer}} } -void test_amdgcn_raw_ptr_buffer_store_b64(i64 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset, int aux) { +void test_amdgcn_raw_ptr_buffer_store_b64(u64 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset, int aux) { __builtin_amdgcn_raw_buffer_store_b64(vdata, rsrc, /*offset=*/0, /*soffset=*/0, aux); //expected-error{{argument to '__builtin_amdgcn_raw_buffer_store_b64' must be a constant integer}} } -void test_amdgcn_raw_ptr_buffer_store_b96(i96 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset, int aux) { +void test_amdgcn_raw_ptr_buffer_store_b96(u96 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset, int aux) { __builtin_amdgcn_raw_buffer_store_b96(vdata, rsrc, /*offset=*/0, /*soffset=*/0, aux); //expected-error{{argument to '__builtin_amdgcn_raw_buffer_store_b96' must be a constant integer}} } -void test_amdgcn_raw_ptr_buffer_store_b128(i128 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset, int aux) { +void test_amdgcn_raw_ptr_buffer_store_b128(u128 vdata, __amdgpu_buffer_rsrc_t rsrc, int offset, int soffset, int aux) { __builtin_amdgcn_raw_buffer_store_b128(vdata, rsrc, /*offset=*/0, /*soffset=*/0, aux); //expected-error{{argument to '__builtin_amdgcn_raw_buffer_store_b128' must be a constant integer}} } _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits