This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PATCH RFA: Better warnings for CONST_DOUBLE
- From: Ian Lance Taylor <ian at airs dot com>
- To: Richard Henderson <rth at redhat dot com>
- Cc: Geoffrey Keating <geoffk at apple dot com>, gcc-patches at gcc dot gnu dot org
- Date: 31 Jul 2005 22:58:58 -0700
- Subject: Re: PATCH RFA: Better warnings for CONST_DOUBLE
- References: <m3vf2vaj16.fsf@gossamer.airs.com><20050728003925.GA13702@redhat.com> <m3iryu98ma.fsf@gossamer.airs.com><20050729055533.GA17775@redhat.com><BB51417A-F221-4D01-BB1E-20D1483D8AC3@apple.com><20050729202448.GC20008@redhat.com>
Richard Henderson <rth@redhat.com> writes:
> On Fri, Jul 29, 2005 at 09:45:47AM -0700, Geoffrey Keating wrote:
> > Did you build with --enable-checking=rtl? It appears that merely --
> > enable-checking does not switch on RTL checking...
>
> Ah, indeed not. I'll restart.
Sorry about that. That was rather a mess-up on my part. I just
assumed that --enable-checking included ENABLE_RTL_CHECKING.
I completed a bootstrap on i386 with Richard's updated patch, plus
this additional patch.
Ian
Index: real.h
===================================================================
RCS file: /cvs/gcc/gcc/gcc/real.h,v
retrieving revision 1.84
diff -p -u -r1.84 real.h
--- real.h 25 Jun 2005 02:00:50 -0000 1.84
+++ real.h 1 Aug 2005 05:58:31 -0000
@@ -364,7 +364,7 @@ REAL_VALUE_TYPE real_value_from_int_cst
/* Given a CONST_DOUBLE in FROM, store into TO the value it represents. */
#define REAL_VALUE_FROM_CONST_DOUBLE(to, from) \
- memcpy (&(to), &CONST_DOUBLE_LOW ((from)), sizeof (REAL_VALUE_TYPE))
+ memcpy (&(to), CONST_DOUBLE_REAL_VALUE ((from)), sizeof (REAL_VALUE_TYPE))
/* Return a CONST_DOUBLE with value R and mode M. */
#define CONST_DOUBLE_FROM_REAL_VALUE(r, m) \