[llvm] [True16] Make loads into s16 legal in GlobalISel (PR #176963)

Domenic Nutile via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 2 08:13:31 PST 2026


================
@@ -662,10 +668,16 @@ define half @test_vector_reduce_fminimum_v8half(<8 x half> %v) {
 ; GFX12-GISEL-TRUE16-NEXT:    s_wait_samplecnt 0x0
 ; GFX12-GISEL-TRUE16-NEXT:    s_wait_bvhcnt 0x0
 ; GFX12-GISEL-TRUE16-NEXT:    s_wait_kmcnt 0x0
-; GFX12-GISEL-TRUE16-NEXT:    v_minimum_f16 v0.l, v0.l, v0.h
-; GFX12-GISEL-TRUE16-NEXT:    v_minimum_f16 v0.h, v1.l, v1.h
-; GFX12-GISEL-TRUE16-NEXT:    v_minimum_f16 v1.l, v2.l, v2.h
-; GFX12-GISEL-TRUE16-NEXT:    v_minimum_f16 v1.h, v3.l, v3.h
+; GFX12-GISEL-TRUE16-NEXT:    v_lshrrev_b32_e32 v4, 16, v0
+; GFX12-GISEL-TRUE16-NEXT:    v_lshrrev_b32_e32 v5, 16, v1
+; GFX12-GISEL-TRUE16-NEXT:    v_lshrrev_b32_e32 v6, 16, v2
+; GFX12-GISEL-TRUE16-NEXT:    v_lshrrev_b32_e32 v7, 16, v3
----------------
saxlungs wrote:

I'm going to resolve this because this change is no longer in the PR due to removing the G_MERGE/UNMERGE_VALUES changes. However, I will be keeping an eye on this after this PR and Petar's PR go in to make sure nothing like this pops up in the tests again

https://github.com/llvm/llvm-project/pull/176963


More information about the llvm-commits mailing list