From patchwork Wed Jul 4 11:04:23 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: Move Graphite from using PPL over to ISL Date: Wed, 04 Jul 2012 01:04:23 -0000 From: Tristan Gingold X-Patchwork-Id: 168950 Message-Id: <70EA4DC5-0808-429F-89E9-C38D147CA287@adacore.com> To: Richard Guenther Cc: Diego Novillo , gcc-patches@gcc.gnu.org, tobias@grosser.es, sebpop@gmail.com, Michael Matz On Jul 4, 2012, at 12:59 PM, Richard Guenther wrote: > On Wed, 4 Jul 2012, Tristan Gingold wrote: > >> >> On Jul 4, 2012, at 12:32 PM, Richard Guenther wrote: >> >>> On Wed, 4 Jul 2012, Tristan Gingold wrote: >>> >>>> >>>> On Jul 4, 2012, at 10:58 AM, Richard Guenther wrote: >>>> >>>>> On Wed, 4 Jul 2012, Tristan Gingold wrote: >>>>> >>>>>> >>>>>> On Jul 2, 2012, at 1:43 PM, Richard Guenther wrote: >>>>>> >>>>>>> On Thu, 28 Jun 2012, Diego Novillo wrote: >>>>>>> >>>>>>>> On 12-06-27 11:06 , Richard Guenther wrote: >>>>>>>> >>>>>>>>> 2012-06-27 Richard Guenther >>>>>>>>> Michael Matz >>>>>>>>> Tobias Grosser >>>>>>>>> Sebastian Pop >>>>>>>>> >>>>>>>>> config/ >>>>>>>>> * cloog.m4: Set up to work against ISL only. >>>>>>>>> * isl.m4: New file. >>>>>> >>>>>> Shouldn't ENABLE_IS_CHECK be automatically set to no when building a cross canadian ? >>>>> >>>>> Probably yes, as it is a runtime test. Thus at any time when host != >>>>> build. Care to post a patch? You probably know best what the canonical >>>>> way to check this is. >>>> >>>> Something like that ? >>>> >>>> Tested by building a cross and a canadian for ia64-hp-openvms. >>> >>> Seems to be an incremental patch ontop of the real patch? >> >> No, what does make you think so ? >> >> That's an incremental patch on top of many VMS specific unrelated patches, but this is the real patch: building a cross compiler fails without it. > > Well, > > diff --git a/config/isl.m4 b/config/isl.m4 > index 152d422..831ac26 100644 > --- a/config/isl.m4 > +++ b/config/isl.m4 > @@ -103,7 +103,7 @@ AC_DEFUN([ISL_REQUESTED], > > # _ISL_CHECK_CT_PROG(MAJOR, MINOR) > # -------------------------------------------- > -# Helper for verifying CLooG's compile time version. > +# Helper for verifying ISL compile time version. > m4_define([_ISL_CHECK_CT_PROG],[AC_LANG_PROGRAM( > [#include > [gcc_cv_isl=yes], > - [gcc_cv_isl=no])]) > + [gcc_cv_isl=no], > + [gcc_cv_isl=yes])]) > > CFLAGS=$_isl_saved_CFLAGS > LDFLAGS=$_isl_saved_LDFLAGS > > Does not apply for me - there seems to be lines missing. I guess > the 2nd hunk is for Arggh, indeed it was truncated. Here is the correct one. Tristan. 2012-07-04 Tristan Gingold * configure: Regenerate. config/ 2012-07-04 Tristan Gingold * isl.m4 (ISL_CHECK_VERSION): Set to yes if cross-compiling. Fix comments. diff --git a/config/isl.m4 b/config/isl.m4 index 152d422..4f39724 100644 --- a/config/isl.m4 +++ b/config/isl.m4 @@ -84,7 +84,7 @@ AC_DEFUN([ISL_INIT_FLAGS], # ISL_REQUESTED (ACTION-IF-REQUESTED, ACTION-IF-NOT) # ---------------------------------------------------- -# Provide actions for failed CLooG detection. +# Provide actions for failed ISL detection. AC_DEFUN([ISL_REQUESTED], [ AC_REQUIRE([ISL_INIT_FLAGS]) @@ -103,7 +103,7 @@ AC_DEFUN([ISL_REQUESTED], # _ISL_CHECK_CT_PROG(MAJOR, MINOR) # -------------------------------------------- -# Helper for verifying CLooG's compile time version. +# Helper for verifying ISL compile time version. m4_define([_ISL_CHECK_CT_PROG],[AC_LANG_PROGRAM( [#include #include ], @@ -131,7 +131,8 @@ AC_DEFUN([ISL_CHECK_VERSION], [gcc_cv_isl], [AC_RUN_IFELSE([_ISL_CHECK_CT_PROG($1,$2)], [gcc_cv_isl=yes], - [gcc_cv_isl=no])]) + [gcc_cv_isl=no], + [gcc_cv_isl=yes])]) CFLAGS=$_isl_saved_CFLAGS LDFLAGS=$_isl_saved_LDFLAGS