On 6/5/22, LIU Hao <[email protected]> wrote: > 在 2022-06-05 19:30, Ozkan Sezer 写道: >> On 6/5/22, LIU Hao <[email protected]> wrote: >>> 在 2022-06-03 22:03, Ozkan Sezer 写道: >>>> >>>> A quick x64 build ran under win10 yielded the same results. >>>> >>> >>> As far as I can tell, `CVTSD2SI` should be preferred to `FISTP` for >>> `float` >>> and `double`. The X87 >>> implementation was added in 2007 and has not been updated since ever. >> >> Should I push changes to lrint.c and lrintf.c, then? >> (Won't be touching llrint.c) > > Those changes look good to me. Please go ahead.
Change pushed as https://sourceforge.net/p/mingw-w64/mingw-w64/ci/568ddf198ac1ab44773482c272c046b75eefd746/ > BTW, I think we can move a bit further: Check for `__SSE2__` instead of > `__x86_64__`. This enables > SSE2 code for x86 when, for example, `-march=pentium4` is enabled, which is > now the default for > MSYS2. It's worth another patch, though. Feel free to do so. -- O.S. _______________________________________________ Mingw-w64-public mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
