[FFmpeg-devel] [PATCH] lavc/h264chroma: RISC-V V add motion compensation for 4xH and 2xH chroma blocks

Arnie Chang arnie.chang at sifive.com
Thu Jun 15 17:58:37 EEST 2023

On Wed, Jun 14, 2023 at 11:57 PM Rémi Denis-Courmont <remi at remlab.net>

> It looks like \width is only ever used as AVL. You could advantageously
> pass
> it as a run-time argument to an internal function, and spare the
> instruction
> cache, instead of instantiating otherwise identical code thrice.

Since these functions are frequently called, I prefer instantiating similar
code many times
rather than calling another internal function, as it may introduce
additional function call overhead.
Neon and MMX code also apply the same approach.

    slli t3, a2, (1 + \unroll)

Brilliant suggestion, I will fix it in the next patch.

More information about the ffmpeg-devel mailing list