On 14 February 2011 16:47, Christophe Lyon <christophe.l...@st.com> wrote:
>
> These two special values are needed to implement some helper functions, which 
> return these values in some cases.

I assume this is for vrecpe, right? I think it would be better
to post this as part of that patchset; it's easier to see the
rationale for adding the functions when you also have the
code that's using them.

>  /*----------------------------------------------------------------------------
> +| Returns the default NaN.
> +*----------------------------------------------------------------------------*/
> +
> +float32 float32_nan(void)
> +{
> +    return float32_default_nan;
> +}

Maybe we should just expose float32_default_nan in
softfloat.h somehow; that would match float32_zero,
float32_one, etc (ie it would not be a function).

> +#define float32_infinity make_float32(0x7f800000)

No objection to this one.

-- PMM

Reply via email to