This is the mail archive of the
gcc@gcc.gnu.org
mailing list for the GCC project.
Re: Your patch to libjava/java/lang/ieeefp.h
- From: Andreas Tobler <toa at pop dot agri dot ch>
- To: Andrew Pinski <pinskia at physics dot uc dot edu>
- Cc: Tom Tromey <tromey at redhat dot com>, gcc at gcc dot gnu dot org, java-patches at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org
- Date: Wed, 02 Jan 2002 11:21:55 +0100
- Subject: Re: Your patch to libjava/java/lang/ieeefp.h
- Organization: zero
- References: <08FA1737-FF31-11D5-8238-000393122612@physics.uc.edu>
Hi Andrew,
Andrew Pinski wrote:
>
> Dear Andreas Tobler,
> Your patch:
> 2001-12-29 Andreas Tobler <a.tobler@schweiz.ch>
>
> * libjava/java/lang/ieeefp.h: Define __IEEE_BIG_ENDIAN for
> Darwin.
> is wrong because Darwin runs on both PPC and ia32.
Yup, I know, a private discussion with Bryce showed me that and I was
working on something better. But you got it first.
Thanks, tested on darwin5.2 (ppc) and powerpc-unknown-linux-gnu.
Andreas
> --- src/libjava/java/lang/ieeefp.h Tue Jan 1 22:26:55 2002
> +++ src/libjava/java/lang/ieeefp.h.new Tue Jan 1 22:26:02 2002
> @@ -96,12 +96,8 @@
> #define __IEEE_LITTLE_ENDIAN
> #endif
>
> -#ifdef __APPLE__
> -#define __IEEE_BIG_ENDIAN
> -#endif
> -
> -#ifdef __PPC__
> -#if (defined(_BIG_ENDIAN) && _BIG_ENDIAN) || (defined(_AIX) && _AIX)
> +#if defined(__PPC__) || defined(__ppc__)
> +#if (defined(_BIG_ENDIAN) && _BIG_ENDIAN) || (defined(_AIX) && _AIX) ||
> (defined(__APPLE__))
> #define __IEEE_BIG_ENDIAN
> #else
> #if (defined(_LITTLE_ENDIAN) && _LITTLE_ENDIAN) || (defined(__sun__) &&
> __sun__) || (defined(__WIN32__) && __WIN32__)