Tobias Grosser wrote:
On 10.11.2014 20:14, Roman Gareev wrote:
Sure. We should drop the flag in these test cases.
This seems to make sense, as they now test something different and
the flag removal would reflect this.
I personally would include this in the same patch. Would this be
difficult?
I don’t think that it could be difficult. I just wanted to reduce the
size of a patch which can be found below.
This LGTM if it passes tests.
Seems as if the next topics would be:
* Removal of CLooG from the main configure.ac, config/cloog.m4,
Makefile.tpl + regenation of configure and Makefile.{in,def} + syncing
with sourceware.org's tree.
* Changes to permit using ISL 0.14.0 – with or without still supporting
0.12.0 (the latter seems to be preferred by Richard as it permits to use
the same system lib also with GCC 4.8/4.9). When/if 0.14 works, one can
also put the new version at infrastructure, which fixes PR62289.
* Mentioning in gcc-5/changes.html (Caveats) that CLooG is no longer
required for graphite. (Cf. https://gcc.gnu.org/gcc-4.8/changes.html)
* Removal of CLooG from install.texi's Prerequisites and from
contrib/download_prerequisites – that's what the attached patch does. I
intent to commit it tomorrow as obvious, if there are no objections.
Tobias
2014-11-11 Tobias Burnus <bur...@net-b.de>
contrib/
* download_prerequisites: Stop downloading CLooG.
gcc/
* doc/install.texi (Prerequisites): Remove CLooG.
diff --git a/contrib/download_prerequisites b/contrib/download_prerequisites
index cc27143..a9a9f02 100755
--- a/contrib/download_prerequisites
+++ b/contrib/download_prerequisites
@@ -44,13 +44,8 @@ ln -sf $MPC mpc || exit 1
# Necessary to build GCC with the Graphite loop optimizations.
if [ "$GRAPHITE_LOOP_OPT" = "yes" ] ; then
ISL=isl-0.12.2
- CLOOG=cloog-0.18.1
wget ftp://gcc.gnu.org/pub/gcc/infrastructure/$ISL.tar.bz2 || exit 1
tar xjf $ISL.tar.bz2 || exit 1
ln -sf $ISL isl || exit 1
-
- wget ftp://gcc.gnu.org/pub/gcc/infrastructure/$CLOOG.tar.gz || exit 1
- tar xzf $CLOOG.tar.gz || exit 1
- ln -sf $CLOOG cloog || exit 1
fi
diff --git a/gcc/doc/install.texi b/gcc/doc/install.texi
index 3df78ff..fa5fe6e 100644
--- a/gcc/doc/install.texi
+++ b/gcc/doc/install.texi
@@ -393,21 +393,6 @@ built together with GCC. Alternatively, the @option{--with-isl} configure
option should be used if ISL is not installed in your default library
search path.
-@item CLooG 0.18.1
-
-Necessary to build GCC with the Graphite loop optimizations. It can be
-downloaded from @uref{ftp://gcc.gnu.org/pub/gcc/infrastructure/} as
-@file{cloog-0.18.1.tar.gz}. If a CLooG source distribution is found
-in a subdirectory of your GCC sources named @file{cloog}, it will be
-built together with GCC. Alternatively, the @option{--with-cloog} configure
-option should be used if CLooG is not installed in your default library search
-path.
-
-If you want to install CLooG separately it needs to be built against
-ISL 0.12.2 by using the @option{--with-isl=system} to direct CLooG to pick
-up an already installed ISL. Using the ISL library as bundled with CLooG
-is not supported.
-
@end table
@heading Tools/packages necessary for modifying GCC