> -----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.h > > On Fri, Dec 21, 2012 at 5:14 AM, Richard Biener <rguent...@suse.de> 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 "unfixed") > > > > Thanks, > > Richard. > > > > 2012-12-21 Richard Biener <rguent...@suse.de> > > > > PR bootstrap/54659 > > * system.h: Include gmp.h. > > * tree-ssa-loop-niter.c: Do not include gmp.h here. > > * double-int.h: Likewise. > > * realmpfr.h: Likewise. > > > > fortran/ > > * gfortran.h: Do not include gmp.h here. > > > > This breaks bootstrap with in-tree gmp since system.h is included for host > feature test by gcc/configure. Before gmp is built, gmp.h isn't available. > We > need to build gmp first before configuring gcc. > > Balaji, can you try this patch?
Yes, this seem to do the trick in my initial runs! Is this already committed to the trunk? If so, I will just merge my branch (cilkplus) with the trunk. Otherwise I have to manually apply it. Thanks, Balaji V. Iyer. > > > -- > H.J. > -- > 2013-01-08 H.J. Lu <hongjiu...@intel.com> > > * Makefile.def (configure-gcc): Depend on all-gmp. > (all-gcc): Remove dependency on all-gmp. > * Makefile.in: Regenerated.
