This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [GSoC] Addition of ISL AST generation to Graphite
- From: Rainer Orth <ro at CeBiTec dot Uni-Bielefeld dot DE>
- To: Roman Gareev <gareevroman at gmail dot com>
- Cc: Tobias Grosser <tobias at grosser dot es>, sebpop at gmail dot com, Mircea Namolaru <mircea dot namolaru at inria dot fr>, gcc-patches at gcc dot gnu dot org
- Date: Sat, 05 Jul 2014 22:20:50 +0200
- Subject: Re: [GSoC] Addition of ISL AST generation to Graphite
- Authentication-results: sourceware.org; auth=none
- References: <CABGF_gcisNL2R2vVd8A1KaG1gcgOUok0EekK=8PTtkxg_VHU5w at mail dot gmail dot com> <53A1E2B9 dot 3090205 at grosser dot es> <CAFk3UF9KO7Jodwno__0xpHcmG_Wij8Y6k3VZoRXreduryCtMgg at mail dot gmail dot com> <53A7F4F6 dot 6000707 at grosser dot es> <CABGF_gfQm8z_avKo15k1=B4CDkxUTVCqXSfv-hXYihHb=9MEAw at mail dot gmail dot com> <53A82B9A dot 5000104 at grosser dot es> <CABGF_gdNu1UF_yepWOZyJby_qCT8WthcZSi7c1DY+WVMgyJhtQ at mail dot gmail dot com> <yddbnt5i8lt dot fsf at lokon dot CeBiTec dot Uni-Bielefeld dot DE>
Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE> writes:
> Roman Gareev <gareevroman@gmail.com> writes:
>
>>> It seems the patch1/patch2 files you attach have the Content-Type:
>>> application/octet-stream. This makes it impossible to view them inline.
>>> Could you send them as text files? Just calling them patch1.patch or
>>> patch1.txt should make this work.
>>
>> Yes, sure.
>
> This patch broke bootstrap with --enable-cloog-backend=isl when using
> isl 0.10:
>
> /vol/gcc/src/hg/trunk/local/gcc/graphite-isl-ast-to-gimple.c:27:27: fatal error: isl/ast_build.h: No such file or directory
> compilation terminated.
> make[3]: *** [graphite-isl-ast-to-gimple.o] Error 1
>
> <isl/ast_build.h> is missing in in isl 0.10. Seems it was introduced in
> 0.11, but no idea if that works.
>
> Toplevel configure.ac accepts 0.10, while install.texi states 0.12.2 is
> required. Configuration and/or docs should be changed to match reality.
I've now commited the following patch as obvious: it requires isl 0.11
which is the first that includes isl/ast_build.h.
Bootstrapped on i386-pc-solaris2.11 with isl 0.12.2.
Rainer
2014-07-05 Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE>
* configure.ac: Don't accept isl 0.10.
* configure: Regenerate.
diff --git a/configure.ac b/configure.ac
--- a/configure.ac
+++ b/configure.ac
@@ -1650,12 +1650,9 @@ if test "x$with_isl" != "xno" &&
dnl with user input.
ISL_INIT_FLAGS
dnl The versions of ISL that work for Graphite
- ISL_CHECK_VERSION(0,10)
+ ISL_CHECK_VERSION(0,11)
if test "${gcc_cv_isl}" = no ; then
- ISL_CHECK_VERSION(0,11)
- if test "${gcc_cv_isl}" = no ; then
- ISL_CHECK_VERSION(0,12)
- fi
+ ISL_CHECK_VERSION(0,12)
fi
dnl Only execute fail-action, if ISL has been requested.
ISL_IF_FAILED([
--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University