May 18, 2023, 10:21 by arnie.ch...@sifive.com: > On Wed, May 17, 2023 at 10:54 PM Lynne <d...@lynne.ee> wrote: > >> >> 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. >> > > The checkasm does not cover motion compensation of chroma samples in the > H.264 decoder, > so my modification can not be tested. >
I added one here http://ffmpeg.org/pipermail/ffmpeg-devel/2023-May/309812.html follow the example and add a test next time. I wouldn't accept risc-v asm without checkasm, the platform is too new (and doesn't even have hardware yet), and probably prone to silicon issues. We've found a few silicon bugs through checkasm over the years. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
