A warning patch for config/sparc/*

Kaveh R. Ghazi ghazi@caip.rutgers.edu
Sat Mar 6 18:54:00 GMT 1999


 > From: Richard Henderson <rth@cygnus.com>
 > 
 > On Tue, Mar 02, 1999 at 11:17:23AM -0500, Kaveh R. Ghazi wrote:
 > > -static char sccsid[] = "@(#)gmon.c	5.3 (Berkeley) 5/22/91";
 > > +static char sccsid[] ATTRIBUTE_UNUSED = "@(#)gmon.c	5.3 (Berkeley) 5/22/91";
 > 
 > If this is retained at all, we should use #ident; that's what it's for.


	I'll take it out.




 > > -          extern etext();
 > > +          extern int etext();
 > >  	  already_setup = 1;
 > > -	  monstartup(0, etext);
 > > +	  monstartup(0, (char *)etext);
 > 
 > Better to declare as `extern char etext[]'.


	Okay.



 > >  #if HOST_BITS_PER_WIDE_INT != 64
 > >        && ((CONST_DOUBLE_HIGH (operands[1]) == 0
 > >  	   && (CONST_DOUBLE_LOW (operands[1]) & 0x80000000) == 0)
 > > -	  || (CONST_DOUBLE_HIGH (operands[1]) == 0xffffffff
 > > +	  || (CONST_DOUBLE_HIGH (operands[1]) == (HOST_WIDE_INT) 0xffffffff
 > >  	      && (CONST_DOUBLE_LOW (operands[1]) & 0x80000000) != 0))
 > >  #endif
 > 
 > You might also change that to `#if HOST_BITS_PER_WIDE_INT == 32'.
 > I don't like presuming 32 and 64 are the only sizes we'll ever see.

	Will do.


 > 
 > The rest is ok.
 > r~

	It turns out that this patch has a problem on SunOS4 cc which is
solved by http://egcs.cygnus.com/ml/egcs-patches/1999-03/msg00085.html . 

Its a short patch and the explanation is contained in that posting, but
when I reread it, it sounded kind of confusing.  If I didn't make sense
let me know.  :-)

Would you please take a moment to approve that one too before I
checkin the sparc changes?

		Thanks,
		--Kaveh
--
Kaveh R. Ghazi			Engagement Manager / Project Services
ghazi@caip.rutgers.edu		Qwest Internet Solutions


More information about the Gcc-patches mailing list