Re: [PATCH] Fix PR54659, include gmp.h from system.h

2013-01-09 Thread Richard Biener
On Tue, 8 Jan 2013, H.J. Lu wrote: > On Fri, Dec 21, 2012 at 5:14 AM, Richard Biener wrote: > > > > After the Ada fix including gmp.h from system.h is trivial > > (by means of double-int.h it was included from almost everywhere > > already) > > > > Bootstrapped on x86_64-unknown-linux-gnu, testin

RE: [PATCH] Fix PR54659, include gmp.h from system.h

2013-01-08 Thread Iyer, Balaji V
> -Original Message- > From: H.J. Lu [mailto:hjl.to...@gmail.com] > Sent: Tuesday, January 08, 2013 1:50 PM > To: Richard Biener; Iyer, Balaji V > Cc: gcc-patches@gcc.gnu.org; Diego Novillo; i...@google.com > Subject: Re: [PATCH] Fix PR54659, include gmp.h from system.

Re: [PATCH] Fix PR54659, include gmp.h from system.h

2013-01-08 Thread H.J. Lu
On Fri, Dec 21, 2012 at 5:14 AM, Richard Biener wrote: > > After the Ada fix including gmp.h from system.h is trivial > (by means of double-int.h it was included from almost everywhere > already) > > Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. > > Ok for trunk? (Go frontend "un

Re: [PATCH] Fix PR54659, include gmp.h from system.h

2012-12-27 Thread Richard Biener
On Sat, Dec 22, 2012 at 12:03 PM, Andreas Schwab wrote: > Richard Biener writes: > >> On Sat, Dec 22, 2012 at 10:49 AM, Andreas Schwab >> wrote: >>> Richard Biener writes: >>> (I suppose somehow gmp.h is pulled in configure tests?) >>> >>> AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[ >>> #includ

Re: [PATCH] Fix PR54659, include gmp.h from system.h

2012-12-22 Thread Richard Sandiford
domi...@lps.ens.fr (Dominique Dhumieres) writes: > The following patch allowed me to proceed for > c,c++,lto,fortran,ada,objc,obj-c++ > up to libada which has the same problem: > > --- ../_clean/gcc/configure 2012-12-20 17:19:54.0 +0100 > +++ ../p_work/gcc/configure 2012-12-21 23:44:46

Re: [PATCH] Fix PR54659, include gmp.h from system.h

2012-12-22 Thread Andreas Schwab
Richard Biener writes: > On Sat, Dec 22, 2012 at 10:49 AM, Andreas Schwab > wrote: >> Richard Biener writes: >> >>> (I suppose somehow gmp.h is pulled in configure tests?) >> >> AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[ >> #include "ansidecl.h" >> #include "system.h" >> #ifdef HAVE_SYS_RESOURCE_H

Re: [PATCH] Fix PR54659, include gmp.h from system.h

2012-12-22 Thread Dominique Dhumieres
The following patch allowed me to proceed for c,c++,lto,fortran,ada,objc,obj-c++ up to libada which has the same problem: --- ../_clean/gcc/configure 2012-12-20 17:19:54.0 +0100 +++ ../p_work/gcc/configure 2012-12-21 23:44:46.0 +0100 @@ -10321,9 +10321,9 @@ $as_echo "#defin

Re: [PATCH] Fix PR54659, include gmp.h from system.h

2012-12-22 Thread Richard Biener
On Sat, Dec 22, 2012 at 10:49 AM, Andreas Schwab wrote: > Richard Biener writes: > >> (I suppose somehow gmp.h is pulled in configure tests?) > > AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[ > #include "ansidecl.h" > #include "system.h" > #ifdef HAVE_SYS_RESOURCE_H > #include > #endif > ]], [[rlim_t l

Re: [PATCH] Fix PR54659, include gmp.h from system.h

2012-12-22 Thread Andreas Schwab
Richard Biener writes: > (I suppose somehow gmp.h is pulled in configure tests?) AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[ #include "ansidecl.h" #include "system.h" #ifdef HAVE_SYS_RESOURCE_H #include #endif ]], [[rlim_t l = 0;]])],[],[AC_DEFINE([rlim_t],[long], [Define to `long' if doesn't define

Re: [PATCH] Fix PR54659, include gmp.h from system.h

2012-12-22 Thread Richard Biener
Gerald Pfeifer wrote: >On Fri, 21 Dec 2012, Dominique Dhumieres wrote: >> I think revision 194665 breaks bootstrap on at least >x86_64-apple-darwin10: > >Same on everything FreeBSD. > >PR 55784 - [4.8 regression] declaration of C function 'const char* >strsignal(int)' conflicts with /usr/include

Re: [PATCH] Fix PR54659, include gmp.h from system.h

2012-12-21 Thread Gerald Pfeifer
On Fri, 21 Dec 2012, Dominique Dhumieres wrote: > I think revision 194665 breaks bootstrap on at least x86_64-apple-darwin10: Same on everything FreeBSD. PR 55784 - [4.8 regression] declaration of C function 'const char* strsignal(int)' conflicts with /usr/include/string.h:112: error: previous

Re: [PATCH] Fix PR54659, include gmp.h from system.h

2012-12-21 Thread Dominique Dhumieres
I think I understand at least part of the problem: (1) I configure gcc with ../work/configure --prefix=/opt/gcc/gcc4.8w --enable-languages=c,c++,fortran,objc,obj-c++,java,ada,lto --with-gmp=/opt/mp --with-system-zlib --with-isl=/opt/mp --enable-lto --enable-plugin --enable-build-with-cxx i.e.,

Re: [PATCH] Fix PR54659, include gmp.h from system.h

2012-12-21 Thread Dominique Dhumieres
> That doesn't make much sense. What do the lines around this look like? I am currently bootstrapping r194675 with revision 194665 reverted. The diff between gcc/auto-host.h with(-)/without(+) r194665 reverted and --enable-checking=release for (+) looks like: --- ../build_w/gcc/auto-host.h 2012

Re: [PATCH] Fix PR54659, include gmp.h from system.h

2012-12-21 Thread Ian Lance Taylor
On Fri, Dec 21, 2012 at 8:26 AM, Dominique Dhumieres wrote: > I think revision 194665 breaks bootstrap on at least x86_64-apple-darwin10: > > g++ -c -g -DIN_GCC -fno-exceptions -fno-rtti -fasynchronous-unwind-tables > -W -Wall -Wno-narrowing -Wwrite-strings -Wcast-qual > -Wmissing-format-att

Re: [PATCH] Fix PR54659, include gmp.h from system.h

2012-12-21 Thread Dominique Dhumieres
I think revision 194665 breaks bootstrap on at least x86_64-apple-darwin10: g++ -c -g -DIN_GCC -fno-exceptions -fno-rtti -fasynchronous-unwind-tables -W -Wall -Wno-narrowing -Wwrite-strings -Wcast-qual -Wmissing-format-attribute -pedantic -Wno-long-long -Wno-variadic-macros -Wno-overlength-s

Re: [PATCH] Fix PR54659, include gmp.h from system.h

2012-12-21 Thread Ian Lance Taylor
On Fri, Dec 21, 2012 at 5:14 AM, Richard Biener wrote: > > After the Ada fix including gmp.h from system.h is trivial > (by means of double-int.h it was included from almost everywhere > already) > > Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. > > Ok for trunk? (Go frontend "un

Re: [PATCH] Fix PR54659, include gmp.h from system.h

2012-12-21 Thread Jakub Jelinek
On Fri, Dec 21, 2012 at 02:14:37PM +0100, Richard Biener wrote: > After the Ada fix including gmp.h from system.h is trivial > (by means of double-int.h it was included from almost everywhere > already) > > Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. > > Ok for trunk? (Go fron

[PATCH] Fix PR54659, include gmp.h from system.h

2012-12-21 Thread Richard Biener
After the Ada fix including gmp.h from system.h is trivial (by means of double-int.h it was included from almost everywhere already) Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Ok for trunk? (Go frontend "unfixed") Thanks, Richard. 2012-12-21 Richard Biener PR b