On Wed, 4 Jul 2012, Tristan Gingold wrote:

> 
> 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  <rguent...@suse.de>
> >>>>>>>>>     Michael Matz  <m...@suse.de>
> >>>>>>>>>     Tobias Grosser <tob...@grosser.es>
> >>>>>>>>>     Sebastian Pop <seb...@gmail.com>
> >>>>>>>>> 
> >>>>>>>>>     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 <isl/version.h>
> >        [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.

;)

Ok.

Thanks,
Richard.

> Tristan.
> 
> 2012-07-04  Tristan Gingold  <ging...@adacore.com>
> 
>       * configure: Regenerate.
> 
> config/
> 2012-07-04  Tristan Gingold  <ging...@adacore.com>
> 
>       * 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 <isl/version.h>
>     #include <string.h>],
> @@ -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
> 
> 

-- 
Richard Guenther <rguent...@suse.de>
SUSE / SUSE Labs
SUSE LINUX Products GmbH - Nuernberg - AG Nuernberg - HRB 16746
GF: Jeff Hawn, Jennifer Guild, Felix Imend

Reply via email to