On 11/8/2024 5:15 AM, Martin Storsjö wrote:
On Thu, 7 Nov 2024, James Almer wrote:Should fix fate failures across different systems. Signed-off-by: James Almer <jamr...@gmail.com> ---This only hides the bug in the dither path for unscaled planar yuv 8bit tohbd and vice-versa.Someone more familiar with the code should check what exactly is going on.LGTM, thanks! Good job in hunting this one down - it seems odd as the exact output value seems stable on each machine (and valgrind doesn't find any nondeterminism), but the output is different on most machines.
The only tests that fail are those using the DITHER_COPY method in planarCopyWrapper() when dither is used, and the only difference compared to the no dither path is accessing the "dithers" static const array defined in the same file.
Maybe compilers are doing something weird when aligning it, or accessing it?
OpenPGP_signature.asc
Description: OpenPGP digital signature
_______________________________________________ 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".
