[FFmpeg-devel] [PATCH] lavc/h264chroma: RISC-V V add motion compensation for 8x8 chroma blocks

Lynne dev at lynne.ee
Fri May 19 21:52:57 EEST 2023


May 19, 2023, 19:16 by remi at remlab.net:

> Le keskiviikkona 17. toukokuuta 2023, 17.54.22 EEST Lynne a écrit :
>
>> Finally, run:
>> make checkasm && ./tests/checkasm/checkasm --bench
>> and report on the timings for both the C and assembly versions.
>> If you've made a mistake somewhere, (forgot to restore stack, or a
>> callee-saved register, or your function produces an incorrect result),
>> checkasm will fail.
>>
>
> To be fair, in this particular case, the stack pointer and saved registers are 
> never used, so the risk of messing those are zero.
>
> checkasm would of course verify that the function does that it is supposed to 
> do, and personally, I have kept off untested functions. But I am not sure if it 
> is fair to require adding test cases whilst other architectures weren't 
> required to have them.
>

Other hardware exists, and even without checkasm, bugs are found
through fate. Right now, the only things that can run this code are FPGAs
and emulators, which are both software, something to which bugs are old friends.


More information about the ffmpeg-devel mailing list