Files
Dark Steve d6d99d9f60 [AMDGPU] Improve max3/min3 formation for tree-structured reductions (#190734)
The existing `performMinMaxCombine` forms `max3` by matching
`max(max(a,b), c) -> max3(a,b,c)`. For tree reductions like
`max(max(a,b), max(c,d))`, this produces `max3(a, b, max(c,d));` placing
`max3` on top. At the next tree level, `max(max3, max3)` cannot combine
because the 3-op opcode (`FMAXIMUM3`) differs from the 2-op opcode
(`FMAXIMUM`).

This patch:
1. Adds a tree combine: `max(max(a,b), max(c,d)) -> max(max3(a,b,c),
d)`, keeping a 2-op node on top that enables further combining.
2. Defers the existing combine when the operand is a tree node whose
children can still be combined, ensuring inner tree levels are optimized
before outer levels consume them.

Deferral is skipped when neither child has a single use, since the inner
combine cannot fire in that case.

Fix: LCOMPILER-1652
2026-04-10 12:16:32 +05:30

382 lines
15 KiB
LLVM

; RUN: llc -mtriple=amdgcn < %s | FileCheck -check-prefixes=GCN,SI %s
; RUN: llc -mtriple=amdgcn -mcpu=tonga < %s | FileCheck -check-prefixes=GCN,VI %s
; RUN: llc -mtriple=amdgcn -mcpu=gfx900 < %s | FileCheck -check-prefixes=GCN,GFX9,GFX9_1250 %s
; RUN: llc -mtriple=amdgcn -mcpu=gfx1250 < %s | FileCheck -check-prefixes=GCN,GFX1250,GFX9_1250 %s
; GCN-LABEL: {{^}}v_test_imin3_slt_i32:
; GCN: v_min3_i32
define amdgpu_kernel void @v_test_imin3_slt_i32(ptr addrspace(1) %out, ptr addrspace(1) %aptr, ptr addrspace(1) %bptr, ptr addrspace(1) %cptr) #0 {
%tid = call i32 @llvm.amdgcn.workitem.id.x()
%gep0 = getelementptr i32, ptr addrspace(1) %aptr, i32 %tid
%gep1 = getelementptr i32, ptr addrspace(1) %bptr, i32 %tid
%gep2 = getelementptr i32, ptr addrspace(1) %cptr, i32 %tid
%outgep = getelementptr i32, ptr addrspace(1) %out, i32 %tid
%a = load i32, ptr addrspace(1) %gep0
%b = load i32, ptr addrspace(1) %gep1
%c = load i32, ptr addrspace(1) %gep2
%icmp0 = icmp slt i32 %a, %b
%i0 = select i1 %icmp0, i32 %a, i32 %b
%icmp1 = icmp slt i32 %i0, %c
%i1 = select i1 %icmp1, i32 %i0, i32 %c
store i32 %i1, ptr addrspace(1) %outgep
ret void
}
; GCN-LABEL: {{^}}v_test_umin3_ult_i32:
; GCN: v_min3_u32
define amdgpu_kernel void @v_test_umin3_ult_i32(ptr addrspace(1) %out, ptr addrspace(1) %aptr, ptr addrspace(1) %bptr, ptr addrspace(1) %cptr) #0 {
%tid = call i32 @llvm.amdgcn.workitem.id.x()
%gep0 = getelementptr i32, ptr addrspace(1) %aptr, i32 %tid
%gep1 = getelementptr i32, ptr addrspace(1) %bptr, i32 %tid
%gep2 = getelementptr i32, ptr addrspace(1) %cptr, i32 %tid
%outgep = getelementptr i32, ptr addrspace(1) %out, i32 %tid
%a = load i32, ptr addrspace(1) %gep0
%b = load i32, ptr addrspace(1) %gep1
%c = load i32, ptr addrspace(1) %gep2
%icmp0 = icmp ult i32 %a, %b
%i0 = select i1 %icmp0, i32 %a, i32 %b
%icmp1 = icmp ult i32 %i0, %c
%i1 = select i1 %icmp1, i32 %i0, i32 %c
store i32 %i1, ptr addrspace(1) %outgep
ret void
}
; GCN-LABEL: {{^}}v_test_umin_umin_umin:
; GCN: v_min3_i32
; GCN: v_min_i32
define amdgpu_kernel void @v_test_umin_umin_umin(ptr addrspace(1) %out, ptr addrspace(1) %aptr, ptr addrspace(1) %bptr, ptr addrspace(1) %cptr) #0 {
%tid = call i32 @llvm.amdgcn.workitem.id.x()
%tid2 = mul i32 %tid, 2
%gep0 = getelementptr i32, ptr addrspace(1) %aptr, i32 %tid
%gep1 = getelementptr i32, ptr addrspace(1) %bptr, i32 %tid
%gep2 = getelementptr i32, ptr addrspace(1) %cptr, i32 %tid
%gep3 = getelementptr i32, ptr addrspace(1) %aptr, i32 %tid2
%gep4 = getelementptr i32, ptr addrspace(1) %bptr, i32 %tid2
%gep5 = getelementptr i32, ptr addrspace(1) %cptr, i32 %tid2
%outgep0 = getelementptr i32, ptr addrspace(1) %out, i32 %tid
%outgep1 = getelementptr i32, ptr addrspace(1) %out, i32 %tid2
%a = load i32, ptr addrspace(1) %gep0
%b = load i32, ptr addrspace(1) %gep1
%c = load i32, ptr addrspace(1) %gep2
%d = load i32, ptr addrspace(1) %gep3
%icmp0 = icmp slt i32 %a, %b
%i0 = select i1 %icmp0, i32 %a, i32 %b
%icmp1 = icmp slt i32 %c, %d
%i1 = select i1 %icmp1, i32 %c, i32 %d
%icmp2 = icmp slt i32 %i0, %i1
%i2 = select i1 %icmp2, i32 %i0, i32 %i1
store i32 %i2, ptr addrspace(1) %outgep1
ret void
}
; GCN-LABEL: {{^}}v_test_umin3_2_uses:
; GCN-NOT: v_min3
define amdgpu_kernel void @v_test_umin3_2_uses(ptr addrspace(1) %out, ptr addrspace(1) %aptr, ptr addrspace(1) %bptr, ptr addrspace(1) %cptr) #0 {
%tid = call i32 @llvm.amdgcn.workitem.id.x()
%tid2 = mul i32 %tid, 2
%gep0 = getelementptr i32, ptr addrspace(1) %aptr, i32 %tid
%gep1 = getelementptr i32, ptr addrspace(1) %bptr, i32 %tid
%gep2 = getelementptr i32, ptr addrspace(1) %cptr, i32 %tid
%gep3 = getelementptr i32, ptr addrspace(1) %aptr, i32 %tid2
%gep4 = getelementptr i32, ptr addrspace(1) %bptr, i32 %tid2
%gep5 = getelementptr i32, ptr addrspace(1) %cptr, i32 %tid2
%outgep0 = getelementptr i32, ptr addrspace(1) %out, i32 %tid
%outgep1 = getelementptr i32, ptr addrspace(1) %out, i32 %tid2
%a = load i32, ptr addrspace(1) %gep0
%b = load i32, ptr addrspace(1) %gep1
%c = load i32, ptr addrspace(1) %gep2
%d = load i32, ptr addrspace(1) %gep3
%icmp0 = icmp slt i32 %a, %b
%i0 = select i1 %icmp0, i32 %a, i32 %b
%icmp1 = icmp slt i32 %c, %d
%i1 = select i1 %icmp1, i32 %c, i32 %d
%icmp2 = icmp slt i32 %i0, %c
%i2 = select i1 %icmp2, i32 %i0, i32 %c
store i32 %i2, ptr addrspace(1) %outgep0
store i32 %i0, ptr addrspace(1) %outgep1
ret void
}
; GCN-LABEL: {{^}}v_test_imin3_slt_i16:
; SI: v_min3_i32
; VI: v_min_i16
; VI: v_min_i16
; GFX9_1250: v_min3_i16
define amdgpu_kernel void @v_test_imin3_slt_i16(ptr addrspace(1) %out, ptr addrspace(1) %aptr, ptr addrspace(1) %bptr, ptr addrspace(1) %cptr) #0 {
%tid = call i32 @llvm.amdgcn.workitem.id.x()
%gep0 = getelementptr i16, ptr addrspace(1) %aptr, i32 %tid
%gep1 = getelementptr i16, ptr addrspace(1) %bptr, i32 %tid
%gep2 = getelementptr i16, ptr addrspace(1) %cptr, i32 %tid
%outgep = getelementptr i16, ptr addrspace(1) %out, i32 %tid
%a = load i16, ptr addrspace(1) %gep0
%b = load i16, ptr addrspace(1) %gep1
%c = load i16, ptr addrspace(1) %gep2
%icmp0 = icmp slt i16 %a, %b
%i0 = select i1 %icmp0, i16 %a, i16 %b
%icmp1 = icmp slt i16 %i0, %c
%i1 = select i1 %icmp1, i16 %i0, i16 %c
store i16 %i1, ptr addrspace(1) %outgep
ret void
}
; GCN-LABEL: {{^}}v_test_umin3_ult_i16:
; SI: v_min3_u32
; VI: v_min_u16
; VI: v_min_u16
; GFX9_1250: v_min3_u16
define amdgpu_kernel void @v_test_umin3_ult_i16(ptr addrspace(1) %out, ptr addrspace(1) %aptr, ptr addrspace(1) %bptr, ptr addrspace(1) %cptr) #0 {
%tid = call i32 @llvm.amdgcn.workitem.id.x()
%gep0 = getelementptr i16, ptr addrspace(1) %aptr, i32 %tid
%gep1 = getelementptr i16, ptr addrspace(1) %bptr, i32 %tid
%gep2 = getelementptr i16, ptr addrspace(1) %cptr, i32 %tid
%outgep = getelementptr i16, ptr addrspace(1) %out, i32 %tid
%a = load i16, ptr addrspace(1) %gep0
%b = load i16, ptr addrspace(1) %gep1
%c = load i16, ptr addrspace(1) %gep2
%icmp0 = icmp ult i16 %a, %b
%i0 = select i1 %icmp0, i16 %a, i16 %b
%icmp1 = icmp ult i16 %i0, %c
%i1 = select i1 %icmp1, i16 %i0, i16 %c
store i16 %i1, ptr addrspace(1) %outgep
ret void
}
; GCN-LABEL: {{^}}v_test_imin3_slt_i8:
; SI: v_min3_i32
; VI: v_min_i16
; VI: v_min_i16
; GFX9_1250: v_min3_i16
define amdgpu_kernel void @v_test_imin3_slt_i8(ptr addrspace(1) %out, ptr addrspace(1) %aptr, ptr addrspace(1) %bptr, ptr addrspace(1) %cptr) #0 {
%tid = call i32 @llvm.amdgcn.workitem.id.x()
%gep0 = getelementptr i8, ptr addrspace(1) %aptr, i32 %tid
%gep1 = getelementptr i8, ptr addrspace(1) %bptr, i32 %tid
%gep2 = getelementptr i8, ptr addrspace(1) %cptr, i32 %tid
%outgep = getelementptr i8, ptr addrspace(1) %out, i32 %tid
%a = load i8, ptr addrspace(1) %gep0
%b = load i8, ptr addrspace(1) %gep1
%c = load i8, ptr addrspace(1) %gep2
%icmp0 = icmp slt i8 %a, %b
%i0 = select i1 %icmp0, i8 %a, i8 %b
%icmp1 = icmp slt i8 %i0, %c
%i1 = select i1 %icmp1, i8 %i0, i8 %c
store i8 %i1, ptr addrspace(1) %outgep
ret void
}
; GCN-LABEL: {{^}}v_test_umin3_ult_i8:
; SI: v_min3_u32
; VI: v_min_u16
; VI: v_min_u16
; GFX9_1250: v_min3_u16
define amdgpu_kernel void @v_test_umin3_ult_i8(ptr addrspace(1) %out, ptr addrspace(1) %aptr, ptr addrspace(1) %bptr, ptr addrspace(1) %cptr) #0 {
%tid = call i32 @llvm.amdgcn.workitem.id.x()
%gep0 = getelementptr i8, ptr addrspace(1) %aptr, i32 %tid
%gep1 = getelementptr i8, ptr addrspace(1) %bptr, i32 %tid
%gep2 = getelementptr i8, ptr addrspace(1) %cptr, i32 %tid
%outgep = getelementptr i8, ptr addrspace(1) %out, i32 %tid
%a = load i8, ptr addrspace(1) %gep0
%b = load i8, ptr addrspace(1) %gep1
%c = load i8, ptr addrspace(1) %gep2
%icmp0 = icmp ult i8 %a, %b
%i0 = select i1 %icmp0, i8 %a, i8 %b
%icmp1 = icmp ult i8 %i0, %c
%i1 = select i1 %icmp1, i8 %i0, i8 %c
store i8 %i1, ptr addrspace(1) %outgep
ret void
}
; GCN-LABEL: {{^}}v_test_imin3_slt_i7:
; SI: v_min3_i32
; VI: v_min_i16
; VI: v_min_i16
; GFX9_1250: v_min3_i16
define amdgpu_kernel void @v_test_imin3_slt_i7(ptr addrspace(1) %out, ptr addrspace(1) %aptr, ptr addrspace(1) %bptr, ptr addrspace(1) %cptr) #0 {
%tid = call i32 @llvm.amdgcn.workitem.id.x()
%gep0 = getelementptr i7, ptr addrspace(1) %aptr, i32 %tid
%gep1 = getelementptr i7, ptr addrspace(1) %bptr, i32 %tid
%gep2 = getelementptr i7, ptr addrspace(1) %cptr, i32 %tid
%outgep = getelementptr i7, ptr addrspace(1) %out, i32 %tid
%a = load i7, ptr addrspace(1) %gep0
%b = load i7, ptr addrspace(1) %gep1
%c = load i7, ptr addrspace(1) %gep2
%icmp0 = icmp slt i7 %a, %b
%i0 = select i1 %icmp0, i7 %a, i7 %b
%icmp1 = icmp slt i7 %i0, %c
%i1 = select i1 %icmp1, i7 %i0, i7 %c
store i7 %i1, ptr addrspace(1) %outgep
ret void
}
; GCN-LABEL: {{^}}v_test_umin3_ult_i7:
; SI: v_min3_u32
; VI: v_min_u16
; VI: v_min_u16
; GFX9_1250: v_min3_u16
define amdgpu_kernel void @v_test_umin3_ult_i7(ptr addrspace(1) %out, ptr addrspace(1) %aptr, ptr addrspace(1) %bptr, ptr addrspace(1) %cptr) #0 {
%tid = call i32 @llvm.amdgcn.workitem.id.x()
%gep0 = getelementptr i7, ptr addrspace(1) %aptr, i32 %tid
%gep1 = getelementptr i7, ptr addrspace(1) %bptr, i32 %tid
%gep2 = getelementptr i7, ptr addrspace(1) %cptr, i32 %tid
%outgep = getelementptr i7, ptr addrspace(1) %out, i32 %tid
%a = load i7, ptr addrspace(1) %gep0
%b = load i7, ptr addrspace(1) %gep1
%c = load i7, ptr addrspace(1) %gep2
%icmp0 = icmp ult i7 %a, %b
%i0 = select i1 %icmp0, i7 %a, i7 %b
%icmp1 = icmp ult i7 %i0, %c
%i1 = select i1 %icmp1, i7 %i0, i7 %c
store i7 %i1, ptr addrspace(1) %outgep
ret void
}
; GCN-LABEL: {{^}}v_test_imin3_slt_i33:
; GCN-NOT: v_min3
define amdgpu_kernel void @v_test_imin3_slt_i33(ptr addrspace(1) %out, ptr addrspace(1) %aptr, ptr addrspace(1) %bptr, ptr addrspace(1) %cptr) #0 {
%tid = call i32 @llvm.amdgcn.workitem.id.x()
%gep0 = getelementptr i33, ptr addrspace(1) %aptr, i32 %tid
%gep1 = getelementptr i33, ptr addrspace(1) %bptr, i32 %tid
%gep2 = getelementptr i33, ptr addrspace(1) %cptr, i32 %tid
%outgep = getelementptr i33, ptr addrspace(1) %out, i32 %tid
%a = load i33, ptr addrspace(1) %gep0
%b = load i33, ptr addrspace(1) %gep1
%c = load i33, ptr addrspace(1) %gep2
%icmp0 = icmp slt i33 %a, %b
%i0 = select i1 %icmp0, i33 %a, i33 %b
%icmp1 = icmp slt i33 %i0, %c
%i1 = select i1 %icmp1, i33 %i0, i33 %c
store i33 %i1, ptr addrspace(1) %outgep
ret void
}
; GCN-LABEL: {{^}}v_test_umin3_ult_i33:
; GCN-NOT: v_min3
define amdgpu_kernel void @v_test_umin3_ult_i33(ptr addrspace(1) %out, ptr addrspace(1) %aptr, ptr addrspace(1) %bptr, ptr addrspace(1) %cptr) #0 {
%tid = call i32 @llvm.amdgcn.workitem.id.x()
%gep0 = getelementptr i33, ptr addrspace(1) %aptr, i32 %tid
%gep1 = getelementptr i33, ptr addrspace(1) %bptr, i32 %tid
%gep2 = getelementptr i33, ptr addrspace(1) %cptr, i32 %tid
%outgep = getelementptr i33, ptr addrspace(1) %out, i32 %tid
%a = load i33, ptr addrspace(1) %gep0
%b = load i33, ptr addrspace(1) %gep1
%c = load i33, ptr addrspace(1) %gep2
%icmp0 = icmp ult i33 %a, %b
%i0 = select i1 %icmp0, i33 %a, i33 %b
%icmp1 = icmp ult i33 %i0, %c
%i1 = select i1 %icmp1, i33 %i0, i33 %c
store i33 %i1, ptr addrspace(1) %outgep
ret void
}
; GCN-LABEL: {{^}}v_test_imin3_slt_i64:
; GCN-NOT: v_min3
define amdgpu_kernel void @v_test_imin3_slt_i64(ptr addrspace(1) %out, ptr addrspace(1) %aptr, ptr addrspace(1) %bptr, ptr addrspace(1) %cptr) #0 {
%tid = call i32 @llvm.amdgcn.workitem.id.x()
%gep0 = getelementptr i64, ptr addrspace(1) %aptr, i32 %tid
%gep1 = getelementptr i64, ptr addrspace(1) %bptr, i32 %tid
%gep2 = getelementptr i64, ptr addrspace(1) %cptr, i32 %tid
%outgep = getelementptr i64, ptr addrspace(1) %out, i32 %tid
%a = load i64, ptr addrspace(1) %gep0
%b = load i64, ptr addrspace(1) %gep1
%c = load i64, ptr addrspace(1) %gep2
%icmp0 = icmp slt i64 %a, %b
%i0 = select i1 %icmp0, i64 %a, i64 %b
%icmp1 = icmp slt i64 %i0, %c
%i1 = select i1 %icmp1, i64 %i0, i64 %c
store i64 %i1, ptr addrspace(1) %outgep
ret void
}
; GCN-LABEL: {{^}}v_test_umin3_ult_i64:
; GCN-NOT: v_min3
define amdgpu_kernel void @v_test_umin3_ult_i64(ptr addrspace(1) %out, ptr addrspace(1) %aptr, ptr addrspace(1) %bptr, ptr addrspace(1) %cptr) #0 {
%tid = call i32 @llvm.amdgcn.workitem.id.x()
%gep0 = getelementptr i64, ptr addrspace(1) %aptr, i32 %tid
%gep1 = getelementptr i64, ptr addrspace(1) %bptr, i32 %tid
%gep2 = getelementptr i64, ptr addrspace(1) %cptr, i32 %tid
%outgep = getelementptr i64, ptr addrspace(1) %out, i32 %tid
%a = load i64, ptr addrspace(1) %gep0
%b = load i64, ptr addrspace(1) %gep1
%c = load i64, ptr addrspace(1) %gep2
%icmp0 = icmp ult i64 %a, %b
%i0 = select i1 %icmp0, i64 %a, i64 %b
%icmp1 = icmp ult i64 %i0, %c
%i1 = select i1 %icmp1, i64 %i0, i64 %c
store i64 %i1, ptr addrspace(1) %outgep
ret void
}
; GCN-LABEL: {{^}}v_test_imin3_slt_v2i16:
; SI-COUNT-2: v_min3_i32
; VI-COUNT-2: v_min_i16
; GFX9-COUNT-2: v_pk_min_i16
; GFX1250: v_pk_min3_i16
define amdgpu_kernel void @v_test_imin3_slt_v2i16(ptr addrspace(1) %out, ptr addrspace(1) %aptr, ptr addrspace(1) %bptr, ptr addrspace(1) %cptr) #0 {
%tid = call i32 @llvm.amdgcn.workitem.id.x()
%gep0 = getelementptr i32, ptr addrspace(1) %aptr, i32 %tid
%gep1 = getelementptr i32, ptr addrspace(1) %bptr, i32 %tid
%gep2 = getelementptr i32, ptr addrspace(1) %cptr, i32 %tid
%outgep = getelementptr <2 x i16>, ptr addrspace(1) %out, i32 %tid
%a = load <2 x i16>, ptr addrspace(1) %gep0
%b = load <2 x i16>, ptr addrspace(1) %gep1
%c = load <2 x i16>, ptr addrspace(1) %gep2
%icmp0 = icmp slt <2 x i16> %a, %b
%i0 = select <2 x i1> %icmp0, <2 x i16> %a, <2 x i16> %b
%icmp1 = icmp slt <2 x i16> %i0, %c
%i1 = select <2 x i1> %icmp1, <2 x i16> %i0, <2 x i16> %c
store <2 x i16> %i1, ptr addrspace(1) %outgep
ret void
}
; GCN-LABEL: {{^}}v_test_imin3_ult_v2i16:
; SI-COUNT-2: v_min3_u32
; VI-COUNT-2: v_min_u16
; GFX9-COUNT-2: v_pk_min_u16
; GFX1250: v_pk_min3_u16
define amdgpu_kernel void @v_test_imin3_ult_v2i16(ptr addrspace(1) %out, ptr addrspace(1) %aptr, ptr addrspace(1) %bptr, ptr addrspace(1) %cptr) #0 {
%tid = call i32 @llvm.amdgcn.workitem.id.x()
%gep0 = getelementptr i32, ptr addrspace(1) %aptr, i32 %tid
%gep1 = getelementptr i32, ptr addrspace(1) %bptr, i32 %tid
%gep2 = getelementptr i32, ptr addrspace(1) %cptr, i32 %tid
%outgep = getelementptr <2 x i16>, ptr addrspace(1) %out, i32 %tid
%a = load <2 x i16>, ptr addrspace(1) %gep0
%b = load <2 x i16>, ptr addrspace(1) %gep1
%c = load <2 x i16>, ptr addrspace(1) %gep2
%icmp0 = icmp ult <2 x i16> %a, %b
%i0 = select <2 x i1> %icmp0, <2 x i16> %a, <2 x i16> %b
%icmp1 = icmp ult <2 x i16> %i0, %c
%i1 = select <2 x i1> %icmp1, <2 x i16> %i0, <2 x i16> %c
store <2 x i16> %i1, ptr addrspace(1) %outgep
ret void
}
declare i32 @llvm.amdgcn.workitem.id.x() #1
attributes #0 = { nounwind }
attributes #1 = { nounwind readnone speculatable }