On Sun, 5 Jun 2022, Ozkan Sezer wrote:
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/
FWIW, the pushed version still passes my tests [1] for this function, so
that's all good!
[1]
https://github.com/mstorsjo/llvm-mingw/blob/20220323/test/crt-test.c#L636-L777
// Martin
_______________________________________________
Mingw-w64-public mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
