[FFmpeg-devel] [PATCH 1/2] tests/checkasm/sw_rgb: Added {yuyv, uyvy}toyuv{420, 422} test cases

[Martin Storsjö]

On Tue, 11 Feb 2025, Krzysztof Pyrkosz via ffmpeg-devel wrote:

> Splitting the previous patch into two.
> I noticed that on my x86 box, one of the newly added tests fail:
>
> MMXEXT:
>   uyvytoyuv420_mmxext (sw_rgb.c:126)
>   yuyvtoyuv420_mmxext (sw_rgb.c:126)
> - sw_rgb.uyvytoyuv          [FAILED]
>
> SSE2, AVX and AVX2 are passing, though.

Hmm, that's problematic - we need to sort that out before we can push this 
test.

It's possible that this is a bug in the mmxext implementation. It's also 
possible that it is a case where the test is overly strict about something 
where we legitimately are allowed to deviate a little. (E.g. for cases 
where we know we have aligned/padded line sizes, it may be acceptable to 
overwrite the payload area by a couple of bytes.)

For failures in mmx tests, it's also possible that the declare_func 
should be a declare_func_emms instead.

We'd need to check exactly what differs in the output.

// Martin