[FFmpeg-devel] [PATCH] lavu/float_dsp: avoid reg-stride in R-V V reverse_fmul

[FFmpeg-devel] [PATCH] lavu/float_dsp: avoid reg-stride in R-V V reverse_fmul

[Rémi Denis-Courmont]

This revectors the inner loop to reverse vectors element in vectors,
thus eliminating the negative register stride. Note that RVV does not
have a vector reverse instruction, so this uses a gather.
---
 libavutil/riscv/float_dsp_rvv.S | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

diff --git a/libavutil/riscv/float_dsp_rvv.S b/libavutil/riscv/float_dsp_rvv.S
index 7cfc890bc2..7e9e84d526 100644
--- a/libavutil/riscv/float_dsp_rvv.S
+++ b/libavutil/riscv/float_dsp_rvv.S
@@ -125,20 +125,25 @@ func ff_vector_fmul_add_rvv, zve32f
         ret
 endfunc
 
+// TODO factor vrsub, separate last iteration?
 // (a0) = (a1) * reverse(a2) [0..a3-1]
 func ff_vector_fmul_reverse_rvv, zve32f
+        vsetvli  t0, zero, e16, m4, ta, ma
         sh2add   a2, a3, a2
-        li       t2, -4 // byte stride
-        addi     a2, a2, -4
+        vid.v    v0
+        vadd.vi  v0, v0, 1
 1:
-        vsetvli  t0, a3, e32, m8, ta, ma
+        vsetvli  t0, a3, e16, m4, ta, ma
         slli     t1, t0, 2
-        vle32.v  v16, (a1)
+        vrsub.vx v4, v0, t0 // v4[i] = [VL-1, VL-2... 1, 0]
+        sub      a2, a2, t1
+        vsetvli  zero, zero, e32, m8, ta, ma
+        vle32.v  v8, (a2)
         sub      a3, a3, t0
-        vlse32.v v24, (a2), t2
+        vle32.v  v16, (a1)
         add      a1, a1, t1
+        vrgatherei16.vv v24, v8, v4 // v24 = reverse(v8)
         vfmul.vv v16, v16, v24
-        sub      a2, a2, t1
         vse32.v  v16, (a0)
         add      a0, a0, t1
         bnez     a3, 1b
-- 
2.42.0

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".