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. > +#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. -- PMM
