From patchwork Mon Nov 10 21:49:26 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Tobias Burnus X-Patchwork-Id: 409092 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 61B0614012C for ; Tue, 11 Nov 2014 08:49:41 +1100 (AEDT) DomainKey-Signature: a=rsa-sha1; c=nofws; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender :message-id:date:from:mime-version:to:cc:subject:references :in-reply-to:content-type; q=dns; s=default; b=uTmh1fDu9sRrkX/gn NPqQ5iP3kpe5j8G2RSbD2J4Rs5xFzmO6B/5k2u1IMFau9GXSDdhgWoQU0TeFpHGj /G7DNXTK9wAROoo5Ru8ebj1iFYWLqdZYC03vB5gcTQmggrlN//Uv3TF7UAtwpvqY QYdkitJQ0z+jbrIWmpMYJSJZEQ= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender :message-id:date:from:mime-version:to:cc:subject:references :in-reply-to:content-type; s=default; bh=x3hmrF+EMULURK19nUT/R/z bMh8=; b=P9WluqZ6kbiy9x/Q6fQKeRJytqc0BzTaPtm8eCsaLzHDtAkLkrBTl7B ZhCG50BjF8EJ3ff8mDxKZThwSinBIDhWg2KNRpAxesL42oKxfm8yE+3YywgfLSnu 1OGER8b1TNH3Vy1wEz1K/VijLYj2cCSLmBLw1mJy7xFmIxSVh6+k= Received: (qmail 20614 invoked by alias); 10 Nov 2014 21:49:34 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Delivered-To: mailing list gcc-patches@gcc.gnu.org Received: (qmail 20602 invoked by uid 89); 10 Nov 2014 21:49:33 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.0 required=5.0 tests=AWL, BAYES_00 autolearn=ham version=3.3.2 X-HELO: mx01.qsc.de Received: from mx01.qsc.de (HELO mx01.qsc.de) (213.148.129.14) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-SHA encrypted) ESMTPS; Mon, 10 Nov 2014 21:49:31 +0000 Received: from tux.net-b.de (port-92-194-240-118.dynamic.qsc.de [92.194.240.118]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mx01.qsc.de (Postfix) with ESMTPSA id 070363CE3E; Mon, 10 Nov 2014 22:49:26 +0100 (CET) Message-ID: <546132E6.9050007@net-b.de> Date: Mon, 10 Nov 2014 22:49:26 +0100 From: Tobias Burnus User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 MIME-Version: 1.0 To: Tobias Grosser , Roman Gareev CC: tobias.burnus@physik.fu-berlin.de, Richard Biener , Albert Cohen , skimo@kotnet.org, Mircea Namolaru , gcc-patches@gcc.gnu.org Subject: Re: RFC: Update ISL under gcc/infrastructure/ ? // Remove CLooG? References: <20141104151755.GA13200@physik.fu-berlin.de> <545AB53E.5080808@grosser.es> <545B1D00.6060605@grosser.es> <545BD2F8.1050104@grosser.es> <545E768A.9000101@grosser.es> <545FA53F.9080304@grosser.es> <5460A8BA.4050003@grosser.es> <546110CE.70504@grosser.es> In-Reply-To: <546110CE.70504@grosser.es> 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 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