[FFmpeg-devel] [PATCH] ac3dsp: RISC-V V float_to_fixed24

[Rémi Denis-Courmont]

Hi,

How did you test it? As per http://ffmpeg.org/pipermail/ffmpeg-devel/2023-June/310720.html we still don't have a FATE instance set up with the RISC-V Vector extension. The only testing consists of my manual runs of checkasm on a K230 board. (We *do* have Zba and Zbb now though, hence the existing extract_exponents()).

Also:
- This does not seem according to the C ABI. AFAIK `unsigned` is sign-extended.
- ALU right before dependent conditional branch should be avoided.
- SHxADD can be used advantageously.


Le 22 novembre 2023 14:00:07 GMT+02:00, flow gg <hlefthleft at gmail.com> a écrit :
>c910
>    float_to_fixed24_c: 208.2
>    float_to_fixed24_rvv_f32: 71.5