This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix bootstrap on Solaris 2.5.1
- From: "Aaron W. LaFramboise" <aaronavay62 at aaronwl dot com>
- To: Eric Botcazou <ebotcazou at libertysurf dot fr>
- Cc: gcc-patches at gcc dot gnu dot org, rth at redhat dot com
- Date: Sun, 26 Sep 2004 13:49:53 -0500
- Subject: Re: [PATCH] Fix bootstrap on Solaris 2.5.1
- References: <200409261553.44316.ebotcazou@libertysurf.fr>
Eric Botcazou wrote:
> Is there any particular reason for 'long'? Both Linux and Solaris want 'int'
> instead. Hence the proposed patch.
I don't think so. I didn't realize that POSIX specified the return type
as int, so I just matched the pre-existing Windows-specific
getpagesize() definition in libgcc2.
That definition also needs to be changed from long to int accordingly.
If this is OK also, could you commit it for me? I have a copyright
assignment but no CVS access.
2004-09-26 Aaron W. LaFramboise <aaronavay62@aaronwl.com>
* libgcc2.c (getpagesize): Return int instead of long.
Index: gcc/gcc/libgcc2.c
===================================================================
RCS file: /cvsroot/gcc/gcc/gcc/libgcc2.c,v
retrieving revision 1.175
diff -c -3 -p -r1.175 libgcc2.c
*** gcc/gcc/libgcc2.c 25 Sep 2004 14:36:37 -0000 1.175
--- gcc/gcc/libgcc2.c 26 Sep 2004 18:41:29 -0000
*************** __enable_execute_stack (void *addr __att
*** 1496,1502 ****
#if defined(WINNT) && ! defined(__CYGWIN__) && ! defined (_UWIN)
! long
getpagesize (void)
{
#ifdef _ALPHA_
--- 1496,1502 ----
#if defined(WINNT) && ! defined(__CYGWIN__) && ! defined (_UWIN)
! int
getpagesize (void)
{
#ifdef _ALPHA_