Skip to content

Commit 65a690d

Browse files
arsenmarsenm
committed
clang/AMDGPU: Emit atomicrmw from {global|flat}_atomic_fadd_v2f16 builtins
1 parent bd298a4 commit 65a690d

File tree

4 files changed

+15
-19
lines changed

4 files changed

+15
-19
lines changed

‎clang/lib/CodeGen/CGBuiltin.cpp‎

Lines changed: 6 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -18655,22 +18655,15 @@ Value *CodeGenFunction::EmitAMDGPUBuiltinExpr(unsigned BuiltinID,
1865518655
Src0 = Builder.CreatePointerBitCastOrAddrSpaceCast(Src0, PTy);
1865618656
return Builder.CreateCall(F, { Src0, Src1, Src2, Src3, Src4 });
1865718657
}
18658-
case AMDGPU::BI__builtin_amdgcn_global_atomic_fadd_v2f16:
1865918658
case AMDGPU::BI__builtin_amdgcn_global_atomic_fmin_f64:
1866018659
case AMDGPU::BI__builtin_amdgcn_global_atomic_fmax_f64:
1866118660
case AMDGPU::BI__builtin_amdgcn_flat_atomic_fadd_f64:
1866218661
case AMDGPU::BI__builtin_amdgcn_flat_atomic_fmin_f64:
1866318662
case AMDGPU::BI__builtin_amdgcn_flat_atomic_fmax_f64:
18664-
case AMDGPU::BI__builtin_amdgcn_flat_atomic_fadd_f32:
18665-
case AMDGPU::BI__builtin_amdgcn_flat_atomic_fadd_v2f16: {
18663+
case AMDGPU::BI__builtin_amdgcn_flat_atomic_fadd_f32: {
1866618664
Intrinsic::ID IID;
1866718665
llvm::Type *ArgTy = llvm::Type::getDoubleTy(getLLVMContext());
1866818666
switch (BuiltinID) {
18669-
case AMDGPU::BI__builtin_amdgcn_global_atomic_fadd_v2f16:
18670-
ArgTy = llvm::FixedVectorType::get(
18671-
llvm::Type::getHalfTy(getLLVMContext()), 2);
18672-
IID = Intrinsic::amdgcn_global_atomic_fadd;
18673-
break;
1867418667
case AMDGPU::BI__builtin_amdgcn_global_atomic_fmin_f64:
1867518668
IID = Intrinsic::amdgcn_global_atomic_fmin;
1867618669
break;
@@ -18690,11 +18683,6 @@ Value *CodeGenFunction::EmitAMDGPUBuiltinExpr(unsigned BuiltinID,
1869018683
ArgTy = llvm::Type::getFloatTy(getLLVMContext());
1869118684
IID = Intrinsic::amdgcn_flat_atomic_fadd;
1869218685
break;
18693-
case AMDGPU::BI__builtin_amdgcn_flat_atomic_fadd_v2f16:
18694-
ArgTy = llvm::FixedVectorType::get(
18695-
llvm::Type::getHalfTy(getLLVMContext()), 2);
18696-
IID = Intrinsic::amdgcn_flat_atomic_fadd;
18697-
break;
1869818686
}
1869918687
llvm::Value *Addr = EmitScalarExpr(E->getArg(0));
1870018688
llvm::Value *Val = EmitScalarExpr(E->getArg(1));
@@ -19085,7 +19073,9 @@ Value *CodeGenFunction::EmitAMDGPUBuiltinExpr(unsigned BuiltinID,
1908519073
case AMDGPU::BI__builtin_amdgcn_ds_atomic_fadd_v2f16:
1908619074
case AMDGPU::BI__builtin_amdgcn_ds_atomic_fadd_v2bf16:
1908719075
case AMDGPU::BI__builtin_amdgcn_global_atomic_fadd_f32:
19088-
case AMDGPU::BI__builtin_amdgcn_global_atomic_fadd_f64: {
19076+
case AMDGPU::BI__builtin_amdgcn_global_atomic_fadd_f64:
19077+
case AMDGPU::BI__builtin_amdgcn_global_atomic_fadd_v2f16:
19078+
case AMDGPU::BI__builtin_amdgcn_flat_atomic_fadd_v2f16: {
1908919079
llvm::AtomicRMWInst::BinOp BinOp;
1909019080
switch (BuiltinID) {
1909119081
case AMDGPU::BI__builtin_amdgcn_atomic_inc32:
@@ -19103,6 +19093,8 @@ Value *CodeGenFunction::EmitAMDGPUBuiltinExpr(unsigned BuiltinID,
1910319093
case AMDGPU::BI__builtin_amdgcn_ds_atomic_fadd_v2bf16:
1910419094
case AMDGPU::BI__builtin_amdgcn_global_atomic_fadd_f32:
1910519095
case AMDGPU::BI__builtin_amdgcn_global_atomic_fadd_f64:
19096+
case AMDGPU::BI__builtin_amdgcn_global_atomic_fadd_v2f16:
19097+
case AMDGPU::BI__builtin_amdgcn_flat_atomic_fadd_v2f16:
1910619098
BinOp = llvm::AtomicRMWInst::FAdd;
1910719099
break;
1910819100
}

‎clang/test/CodeGenOpenCL/builtins-fp-atomics-gfx12.cl‎

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,8 @@ void test_local_add_2f16_noret(__local half2 *addr, half2 x) {
4848
}
4949

5050
// CHECK-LABEL: test_flat_add_2f16
51-
// CHECK: call <2 x half> @llvm.amdgcn.flat.atomic.fadd.v2f16.p0.v2f16(ptr %{{.*}}, <2 x half> %{{.*}})
51+
// CHECK: [[RMW:%.+]] = atomicrmw fadd ptr %{{.+}}, <2 x half> %{{.+}} syncscope("agent") seq_cst, align 4, !amdgpu.no.fine.grained.memory !{{[0-9]+$}}
52+
5253
// GFX12-LABEL: test_flat_add_2f16
5354
// GFX12: flat_atomic_pk_add_f16
5455
half2 test_flat_add_2f16(__generic half2 *addr, half2 x) {
@@ -64,7 +65,8 @@ short2 test_flat_add_2bf16(__generic short2 *addr, short2 x) {
6465
}
6566

6667
// CHECK-LABEL: test_global_add_half2
67-
// CHECK: call <2 x half> @llvm.amdgcn.global.atomic.fadd.v2f16.p1.v2f16(ptr addrspace(1) %{{.*}}, <2 x half> %{{.*}})
68+
// CHECK: [[RMW:%.+]] = atomicrmw fadd ptr addrspace(1) %{{.+}}, <2 x half> %{{.+}} syncscope("agent") seq_cst, align 4, !amdgpu.no.fine.grained.memory !{{[0-9]+$}}
69+
6870
// GFX12-LABEL: test_global_add_half2
6971
// GFX12: global_atomic_pk_add_f16 v2, v[0:1], v2, off th:TH_ATOMIC_RETURN
7072
void test_global_add_half2(__global half2 *addr, half2 x) {
@@ -73,7 +75,8 @@ void test_global_add_half2(__global half2 *addr, half2 x) {
7375
}
7476

7577
// CHECK-LABEL: test_global_add_half2_noret
76-
// CHECK: call <2 x half> @llvm.amdgcn.global.atomic.fadd.v2f16.p1.v2f16(ptr addrspace(1) %{{.*}}, <2 x half> %{{.*}})
78+
// CHECK: [[RMW:%.+]] = atomicrmw fadd ptr addrspace(1) %{{.+}}, <2 x half> %{{.+}} syncscope("agent") seq_cst, align 4, !amdgpu.no.fine.grained.memory !{{[0-9]+$}}
79+
7780
// GFX12-LABEL: test_global_add_half2_noret
7881
// GFX12: global_atomic_pk_add_f16 v[0:1], v2, off
7982
void test_global_add_half2_noret(__global half2 *addr, half2 x) {

‎clang/test/CodeGenOpenCL/builtins-fp-atomics-gfx90a.cl‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ void test_global_add_f64(__global double *addr, double x) {
1818
}
1919

2020
// CHECK-LABEL: test_global_add_half2
21-
// CHECK: call <2 x half> @llvm.amdgcn.global.atomic.fadd.v2f16.p1.v2f16(ptr addrspace(1) %{{.*}}, <2 x half> %{{.*}})
21+
// CHECK: = atomicrmw fadd ptr addrspace(1) %{{.+}}, <2 x half> %{{.+}} syncscope("agent") seq_cst, align 4, !amdgpu.no.fine.grained.memory !{{[0-9]+$}}
2222
// GFX90A-LABEL: test_global_add_half2
2323
// GFX90A: global_atomic_pk_add_f16 v2, v[0:1], v2, off glc
2424
void test_global_add_half2(__global half2 *addr, half2 x) {

‎clang/test/CodeGenOpenCL/builtins-fp-atomics-gfx940.cl‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,8 @@ half2 test_flat_add_f32(__generic float *addr, float x) {
1818
}
1919

2020
// CHECK-LABEL: test_flat_add_2f16
21-
// CHECK: call <2 x half> @llvm.amdgcn.flat.atomic.fadd.v2f16.p0.v2f16(ptr %{{.*}}, <2 x half> %{{.*}})
21+
// CHECK: [[RMW:%.+]] = atomicrmw fadd ptr %{{.+}}, <2 x half> %{{.+}} syncscope("agent") seq_cst, align 4, !amdgpu.no.fine.grained.memory !{{[0-9]+$}}
22+
2223
// GFX940-LABEL: test_flat_add_2f16
2324
// GFX940: flat_atomic_pk_add_f16
2425
half2 test_flat_add_2f16(__generic half2 *addr, half2 x) {

0 commit comments

Comments
 (0)