This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Move Graphite from using PPL over to ISL
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 <rguenther@suse.de>
>>>>>>>>> Michael Matz <matz@suse.de>
>>>>>>>>> Tobias Grosser <tobias@grosser.es>
>>>>>>>>> Sebastian Pop <sebpop@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.
Tristan.
2012-07-04 Tristan Gingold <gingold@adacore.com>
* configure: Regenerate.
config/
2012-07-04 Tristan Gingold <gingold@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