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

["Iyer, Balaji V" <balaji.v.iyer@intel.com>]

> -----Original Message-----
> From: H.J. Lu [mailto:hjl.tools@gmail.com]
> Sent: Tuesday, January 08, 2013 1:50 PM
> To: Richard Biener; Iyer, Balaji V
> Cc: gcc-patches@gcc.gnu.org; Diego Novillo; iant@google.com
> Subject: Re: [PATCH] Fix PR54659, include gmp.h from system.h
> 
> On Fri, Dec 21, 2012 at 5:14 AM, Richard Biener <rguenther@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  <rguenther@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.lu@intel.com>
> 
> 	* Makefile.def (configure-gcc): Depend on all-gmp.
> 	(all-gcc): Remove dependency on all-gmp.
> 	* Makefile.in: Regenerated.