On 16.02.2011 17:54, Peter Maydell wrote: > On 16 February 2011 14:51, <christophe.l...@st.com> wrote: >> From: Christophe Lyon <christophe.l...@st.com> >> >> These two special values are needed to implement some helper >> functions, which return these values in some cases. >> >> This patch also moves the definitions of default_nan for 16, 64, x80 >> and 128 bits floats for consistency with float32. > > Your other patch only uses float32_infinity, not float32_nan > or float32_default_nan, which renders a lot of this patch moot > at the moment.
Indeed, I have another patch for vrsqrte which does need float32_nan. >> +#define float32_nan float32_default_nan > > If we do need to expose NaN, we should just have callers > use float32_default_nan, there's no need for this extra > #define. >
