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

[Arnie Chang]

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

> 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.