Re: fdlibm patch

2007-09-27 Thread Christian Thalinger
On Thu, 2007-09-27 at 19:26 +0200, Robert Schuster wrote:
> Hi,
> the OE guys still keep this little patch. Does anyone know if it is
> correct and should be applied?

Hmm, maybe Michael knows...

> --- classpath/native/fdlibm/ieeefp.h.orig   2006-04-14
> 22:33:09.0 -0400
> +++ classpath/native/fdlibm/ieeefp.h2006-04-14 22:41:46.0
> -0400
> @@ -13,7 +13,7 @@
> byte ordering was big or little endian depending upon the
> target.  
> Modern floating-point formats are naturally ordered; in this case
> __VFP_FP__ will be defined, even if soft-float.  */
> -#ifdef __VFP_FP__
> +#ifdef __SOFTFP__
>  #ifdef __ARMEL__
>  #define __IEEE_LITTLE_ENDIAN
>  #else

- twisti



Re: fdlibm patch

2007-09-27 Thread Robert Lougher
On 9/27/07, Christian Thalinger <[EMAIL PROTECTED]> wrote:
> On Thu, 2007-09-27 at 19:26 +0200, Robert Schuster wrote:
> > Hi,
> > the OE guys still keep this little patch. Does anyone know if it is
> > correct and should be applied?
>

On ARM, soft floating point and the VFP coprocessor use "pure endian"
doubles rather than the original "mixed endian" doubles.

The comment implies __VFP_FP__ should be defined when __SOFTFP__ is
defined.  However, the change implies the comment is wrong because
it's unlikely it would have been changed for the hell of it.

I'd be inclined to play on the safe side and check for either.  This
is what I do in similar places in JamVM :

#if defined(__VFP_FP__) || defined(__SOFTFP__)

Rob.

> Hmm, maybe Michael knows...
>
> > --- classpath/native/fdlibm/ieeefp.h.orig   2006-04-14
> > 22:33:09.0 -0400
> > +++ classpath/native/fdlibm/ieeefp.h2006-04-14 22:41:46.0
> > -0400
> > @@ -13,7 +13,7 @@
> > byte ordering was big or little endian depending upon the
> > target.
> > Modern floating-point formats are naturally ordered; in this case
> > __VFP_FP__ will be defined, even if soft-float.  */
> > -#ifdef __VFP_FP__
> > +#ifdef __SOFTFP__
> >  #ifdef __ARMEL__
> >  #define __IEEE_LITTLE_ENDIAN
> >  #else
>
> - twisti
>
>