On 16 January 2012 00:46, Andreas Färber <afaer...@suse.de> wrote:
> normalizeFloat{32,64}Subnormal() expect the exponent as int16, not int.
> This went unnoticed since int16 and uint16 were both typedef'ed to int.

I think at the time I wrote that I was being slightly conservative
because I didn't feel like confirming that we wouldn't overflow a
16 bit type.

Anyway, I have tested this patch:
 * in combination with your others
 * with typedefs set up so all int16 &co are the minimum width they
   could be
 * with typedefs so int16 &co are all 64 bits wide

and this all still gives the right results for ARM vfm, so

Reviewed-by: Peter Maydell <peter.mayd...@linaro.org>

-- PMM

Reply via email to