|
From: | Richard Henderson |
Subject: | Re: [Qemu-devel] [PATCH 03/22] softfloat: Add 16 bit integer to float conversions |
Date: | Tue, 31 Dec 2013 06:21:22 -0800 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.2.0 |
On 12/31/2013 05:35 AM, Peter Maydell wrote: > +/* We provide the int16 versions for symmetry of API with float-to-int */ > +INLINE float32 int16_to_float32(int_fast16_t v STATUS_PARAM) > +{ > + return int32_to_float32(v STATUS_VAR); > +} If you're going to have int16 versions, I don't think you should use int_fast16_t, but rather int16_t so that we will properly truncate the incoming value. Otherwise there's not much point in having these, IMO. And please add blank lines between the functions. r~
[Prev in Thread] | Current Thread | [Next in Thread] |