On 04/13/2018 04:07 PM, Peter Maydell wrote: > On 13 April 2018 at 15:06, Bastian Koppelmann > <kbast...@mail.uni-paderborn.de> wrote: >> On 04/13/2018 04:03 PM, Alex Bennée wrote: >>> The re-factor broke the raising of INVALID when NaN/Inf is passed to >>> the float_to_int conversion functions. round_to_uint_and_pack got this >>> right for NaN but also missed out the Inf handling. >>> >>> Fixes https://bugs.launchpad.net/qemu/+bug/1759264 >>> >>> Signed-off-by: Alex Bennée <alex.ben...@linaro.org> >>> Cc: Bastian Koppelmann <kbast...@mail.uni-paderborn.de> >>> --- >>> fpu/softfloat.c | 3 +++ >>> 1 file changed, 3 insertions(+) >>> >> >> Reviewed-by: Bastian Koppelmann <kbast...@mail.uni-paderborn.de> > > Could you run your tricore test case with this patch? I think > Alex was having difficulty getting that running...
This fixes the ftoi problem, so Tested-by: Bastian Koppelmann <kbast...@mail.uni-paderborn.de> However, my risu-like tests found another flag raising problem with float32_div. I'll investigate it on Monday. Cheers, Bastian