diff mbox

RFC: Update ISL under gcc/infrastructure/ ? // Remove CLooG?

Message ID 546132E6.9050007@net-b.de
State New
Headers show

Commit Message

Tobias Burnus Nov. 10, 2014, 9:49 p.m. UTC 
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
diff mbox

Patch

2014-11-11  Tobias Burnus  <burnus@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