Re: [FFmpeg-devel] [PATCH] libswscale/x86/yuv2rgb: Add missing EMMS

2024-06-25 Thread Michael Niedermayer
On Tue, Jun 11, 2024 at 05:19:06PM +, Mario Hros wrote: > Ping? > > Not including EMMS causes the FPU to become unusable (as the IE and SF bits > are set in the status word). This breaks code which, for instance, calls > fmod() and is compiled by GCC with the -O3 flag enabled. In that scenar

Re: [FFmpeg-devel] [PATCH] libswscale/x86/yuv2rgb: Add missing EMMS

2024-06-11 Thread Mario Hros
Ping? Not including EMMS causes the FPU to become unusable (as the IE and SF bits are set in the status word). This breaks code which, for instance, calls fmod() and is compiled by GCC with the -O3 flag enabled. In that scenario, GCC implements fmod using the FPREM FPU instruction, but because

[FFmpeg-devel] [PATCH] libswscale/x86/yuv2rgb: Add missing EMMS

2024-06-05 Thread Mario Hros
Previous rewrite from inline assembly into nasm (commit e934194) missed the required EMMS instruction to bring the x87 FPU back into usable state. Signed-off-by: Mario Hros --- libswscale/x86/yuv_2_rgb.asm | 1 + 1 file changed, 1 insertion(+) diff --git a/libswscale/x86/yuv_2_rgb.asm b/libsws