[llvm] [DAGCombiner] Optimize more vector element extractions. (PR #80520)

Harald van Dijk via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 4 09:54:56 PST 2024


================
@@ -190,10 +190,6 @@ define <2 x half> @v_test_canonicalize_build_vector_v2f16(half %lo, half %hi) #1
 ; CI-LABEL: v_test_canonicalize_build_vector_v2f16:
 ; CI:       ; %bb.0:
 ; CI-NEXT:    s_waitcnt vmcnt(0) expcnt(0) lgkmcnt(0)
-; CI-NEXT:    v_cvt_f16_f32_e32 v0, v0
-; CI-NEXT:    v_cvt_f16_f32_e32 v1, v1
-; CI-NEXT:    v_cvt_f32_f16_e32 v0, v0
-; CI-NEXT:    v_cvt_f32_f16_e32 v1, v1
 ; CI-NEXT:    s_setpc_b64 s[30:31]
----------------
hvdijk wrote:

As far as I can see, we were previously converting the `f16` parameters to `f32` and straight back to `f16`. There is no reason for doing that, so I think it is a good thing that this is getting removed now.

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


More information about the llvm-commits mailing list