From: Alexandre Julliard Subject: Re: [PATCH] msvcrt: Don't set MXCSR in _control87. Message-Id: <87eept5atd.fsf@wine> Date: Fri, 03 Jul 2020 09:15:10 +0200 In-Reply-To: (Myah Caron's message of "Thu, 02 Jul 2020 21:57:02 +0000") References: Myah Caron writes: > Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=27594 > Signed-off-by: Myah Caron > --- > This appears to match native MSVCRT's behavior, at least the one provided by vcrun6. It's not obvious because the tests are in kernel32, but we are definitely supposed to set it: ../../../tools/runtest -q -P wine -T ../../.. -M kernel32.dll -p kernel32_test.exe thread && touch thread.ok thread.c:1915: Test failed: expected 0x20301 got 0x80020301 thread.c:1916: Test failed: expected 0x70000c60 got 0x1f800c60 thread.c:1926: Test failed: expected 0x20301 got 0x80020301 thread.c:1927: Test failed: expected 0x70000c60 got 0x1f800c60 make: *** [Makefile:948: thread.ok] Error 4 Looking at your test case, it may be that we shouldn't set it when the FPU control word didn't change. You should extend the tests in thread.c. -- Alexandre Julliard julliard@winehq.org