This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Move Graphite from using PPL over to ISL


On Thu, 5 Jul 2012, Richard Guenther wrote:

> On Thu, 5 Jul 2012, Tristan Gingold wrote:
> 
> > 
> > On Jul 4, 2012, at 8:31 PM, Magnus Fromreide wrote:
> > 
> > > Hello.
> > > 
> > > Ever since the ISL patch went in my builds have failed.
> > 
> > Which ISL patch ?
> > 
> > > I am building with local copies of all the libraries, so I have added
> > > gmp, mpfr, mpc, isl and cloog from ftp://gcc.gnu.org/pub/gcc/infrastructure
> > > to my source directory.
> > 
> > I am not familiar with this build (I usually build gmp/mpfr/... once and install them).
> > But it looks like isl doesn't see the gmp includes.  But according to $(srcdir)/Makefile.in,
> > gmp is configured with --with-gmp-include.  Was isl correctly configured in your case ?
> 
> I have not tested builds with gmp/mpfr in the source tree but have
> copied the logic from the old ppl/cloog configury.  Can you try
> to replace $$r/$(HOST_SUBDIR)/gmp with $$s/gmp in Makefile.def?
> The issue is we can't specify both source and build dirs here
> (so the underlying issue is packages expect installed gmp but we don't
> actually install it - they are not expecting to be used as-is in
> their build directory).

Looking at isl/cloog configure in detail it seems that the following might
work (tested with all reqs in tree only yet).  Hopefully it autodetects
the case where the build dir does not exist and uses the system library
instead then (of course alternate gmp install dirs are still not properly
passed down I guess).

Richard.

Index: Makefile.def
===================================================================
--- Makefile.def        (revision 189263)
+++ Makefile.def        (working copy)
@@ -62,12 +62,12 @@ host_modules= { module= mpc; lib_path=.l
                extra_configure_flags='--disable-shared 
@extra_mpc_gmp_configure_flags@ @extra_mpc_mpfr_configure_flags@';
                no_install= true; };
 host_modules= { module= isl; lib_path=.libs; bootstrap=true;
-               extra_configure_flags='--disable-shared 
--with-gmp-library=$$r/$(HOST_SUBDIR)/gmp/.libs 
--with-gmp-include=$$r/$(HOST_SUBDIR)/gmp --with-bits=gmp';
+               extra_configure_flags='--disable-shared 
--with-gmp-builddir=$$r/$(HOST_SUBDIR)/gmp --with-bits=gmp';
+               extra_make_flags='V=1';
                no_install= true; };
 host_modules= { module= cloog; lib_path=.libs; bootstrap=true;
-               extra_configure_flags='--disable-shared 
--with-gmp-library=$$r/$(HOST_SUBDIR)/gmp/.libs 
--with-gmp-include=$$r/$(HOST_SUBDIR)/gmp --with-bits=gmp 
--with-isl=system';
-               extra_exports='CPPFLAGS="-I$$r/$(HOST_SUBDIR)/isl/include 
-I$$s/isl/include $$CPPFLAGS"; export CPPFLAGS; 
LDFLAGS="-L$$r/$(HOST_SUBDIR)/isl/.libs $$LDFLAGS"; export LDFLAGS; ';
-               extra_make_flags='CPPFLAGS="$$CPPFLAGS" 
LDFLAGS="$$LDFLAGS" V=1';
+               extra_configure_flags='--disable-shared 
--with-gmp-builddir=$$r/$(HOST_SUBDIR)/gmp 
--with-isl-builddir=$$r/$(HOST_SUBDIR)/isl --with-bits=gmp';
+               extra_make_flags='V=1';
                no_install= true; };
 host_modules= { module= libelf; lib_path=.libs; bootstrap=true;
                extra_configure_flags='--disable-shared';


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]