Patchwork [05/28] Document availability of cloog-0.16

login
register
mail settings
Submitter Sebastian Pop
Date Jan. 21, 2011, 7:27 p.m.
Message ID <1295638067-13460-6-git-send-email-sebpop@gmail.com>
Download mbox | patch
Permalink /patch/79889/
State New
Headers show

Comments

Sebastian Pop - Jan. 21, 2011, 7:27 p.m.
From: grosser <grosser@138bc75d-0d04-0410-961f-82ee72b054a4>

2011-01-13 Tobias Grosser <grosser@fim.uni-passau.de>

	* doc/install.texi: Document availability of cloog-0.16.

git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/branches/graphite@168771 138bc75d-0d04-0410-961f-82ee72b054a4
---
 gcc/ChangeLog          |    4 ++++
 gcc/ChangeLog.graphite |    4 ++++
 gcc/doc/install.texi   |   27 +++++++++++++++++----------
 3 files changed, 25 insertions(+), 10 deletions(-)
Gerald Pfeifer - Jan. 24, 2011, 12:03 a.m.
On Fri, 21 Jan 2011, Sebastian Pop wrote:
> --- a/gcc/ChangeLog
> +++ b/gcc/ChangeLog
> @@ -1,3 +1,7 @@
> +2011-01-13  Tobias Grosser  <grosser@fim.uni-passau.de>
> +
> +	* doc/install.texi: Document availability of cloog-0.16.
> +
>  2011-01-21  Vladimir Kargov  <kargov@gmail.com>
>  
>  	* graphite-scop-detection.c (canonicalize_loop_closed_ssa): Free
> diff --git a/gcc/ChangeLog.graphite b/gcc/ChangeLog.graphite
> index d180612..5839242 100644
> --- a/gcc/ChangeLog.graphite
> +++ b/gcc/ChangeLog.graphite
> @@ -1,3 +1,7 @@
> +2011-01-13  Tobias Grosser  <grosser@fim.uni-passau.de>
> +
> +	* doc/install.texi: Document availability of cloog-0.16
> +

Are you really keeping duplicate ChangeLog entries?

Apart from this, it looks like the patch we went through recnetly, 
and I approve with two changes:

> +@item CLooG-PPL version 0.15 / CLooG 0.16

Can you make this an "or" instead of "/"?

> +@uref{ftp://gcc.gnu.org/pub/gcc/infrastructure/} as
> +@file{cloog-0.16.x.tar.gz}.

Can you use the actual filename here?

Gerald
Tobias Grosser - Jan. 24, 2011, 3:11 a.m.
On 01/23/2011 07:03 PM, Gerald Pfeifer wrote:
> On Fri, 21 Jan 2011, Sebastian Pop wrote:
>> --- a/gcc/ChangeLog
>> +++ b/gcc/ChangeLog
>> @@ -1,3 +1,7 @@
>> +2011-01-13  Tobias Grosser<grosser@fim.uni-passau.de>
>> +
>> +	* doc/install.texi: Document availability of cloog-0.16.
>> +
>>   2011-01-21  Vladimir Kargov<kargov@gmail.com>
>>
>>   	* graphite-scop-detection.c (canonicalize_loop_closed_ssa): Free
>> diff --git a/gcc/ChangeLog.graphite b/gcc/ChangeLog.graphite
>> index d180612..5839242 100644
>> --- a/gcc/ChangeLog.graphite
>> +++ b/gcc/ChangeLog.graphite
>> @@ -1,3 +1,7 @@
>> +2011-01-13  Tobias Grosser<grosser@fim.uni-passau.de>
>> +
>> +	* doc/install.texi: Document availability of cloog-0.16
>> +
>
> Are you really keeping duplicate ChangeLog entries?
>
> Apart from this, it looks like the patch we went through recnetly,
> and I approve with two changes:
>
>> +@item CLooG-PPL version 0.15 / CLooG 0.16
>
> Can you make this an "or" instead of "/"?
>
>> +@uref{ftp://gcc.gnu.org/pub/gcc/infrastructure/} as
>> +@file{cloog-0.16.x.tar.gz}.
>
> Can you use the actual filename here?

Sorry Gerald,

it seems I forgot those changes. I just fixed this on the graphite 
branch, such that it is included in the next merge from graphite to trunk.

Thanks for rechecking my commit
Tobi
Gerald Pfeifer - Jan. 26, 2011, 10:43 p.m.
On Sun, 23 Jan 2011, Tobias Grosser wrote:
> it seems I forgot those changes. I just fixed this on the graphite 
> branch, such that it is included in the next merge from graphite to 
> trunk.

No worries, just let's make sure this goes into HEAD before the
release branches. :-)

Gerald
Tobias Grosser - Jan. 26, 2011, 10:44 p.m.
On 01/26/2011 05:43 PM, Gerald Pfeifer wrote:
> On Sun, 23 Jan 2011, Tobias Grosser wrote:
>> it seems I forgot those changes. I just fixed this on the graphite
>> branch, such that it is included in the next merge from graphite to
>> trunk.
>
> No worries, just let's make sure this goes into HEAD before the
> release branches. :-)

OK. I will keep an eye on this, but the next merge should be pretty soon.

Cheers
Tobi
Sebastian Pop - Jan. 26, 2011, 10:48 p.m.
On Wed, Jan 26, 2011 at 16:44, Tobias Grosser <grosser@fim.uni-passau.de> wrote:
> On 01/26/2011 05:43 PM, Gerald Pfeifer wrote:
>>
>> On Sun, 23 Jan 2011, Tobias Grosser wrote:
>>>
>>> it seems I forgot those changes. I just fixed this on the graphite
>>> branch, such that it is included in the next merge from graphite to
>>> trunk.
>>
>> No worries, just let's make sure this goes into HEAD before the
>> release branches. :-)
>
> OK. I will keep an eye on this, but the next merge should be pretty soon.

Tobias, I committed your changes (except Changelog.graphite) to trunk.

Sebastian
Tobias Grosser - Jan. 26, 2011, 10:49 p.m.
On 01/26/2011 05:48 PM, Sebastian Pop wrote:
> On Wed, Jan 26, 2011 at 16:44, Tobias Grosser<grosser@fim.uni-passau.de>  wrote:
>> On 01/26/2011 05:43 PM, Gerald Pfeifer wrote:
>>>
>>> On Sun, 23 Jan 2011, Tobias Grosser wrote:
>>>>
>>>> it seems I forgot those changes. I just fixed this on the graphite
>>>> branch, such that it is included in the next merge from graphite to
>>>> trunk.
>>>
>>> No worries, just let's make sure this goes into HEAD before the
>>> release branches. :-)
>>
>> OK. I will keep an eye on this, but the next merge should be pretty soon.
>
> Tobias, I committed your changes (except Changelog.graphite) to trunk.
>
> Sebastian

Perfect. Thanks

Tobi

Patch

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index cea9b21..9f111e7 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,7 @@ 
+2011-01-13  Tobias Grosser  <grosser@fim.uni-passau.de>
+
+	* doc/install.texi: Document availability of cloog-0.16.
+
 2011-01-21  Vladimir Kargov  <kargov@gmail.com>
 
 	* graphite-scop-detection.c (canonicalize_loop_closed_ssa): Free
diff --git a/gcc/ChangeLog.graphite b/gcc/ChangeLog.graphite
index d180612..5839242 100644
--- a/gcc/ChangeLog.graphite
+++ b/gcc/ChangeLog.graphite
@@ -1,3 +1,7 @@ 
+2011-01-13  Tobias Grosser  <grosser@fim.uni-passau.de>
+
+	* doc/install.texi: Document availability of cloog-0.16
+
 2011-01-05  Vladimir Kargov  <kargov@gmail.com>
 
 	* graphite-scop-detection.c (is_valid_stmt_p): Pass the entry basic
diff --git a/gcc/doc/install.texi b/gcc/doc/install.texi
index 3546170..7d0fa88 100644
--- a/gcc/doc/install.texi
+++ b/gcc/doc/install.texi
@@ -340,16 +340,23 @@  It can be downloaded from @uref{http://www.cs.unipr.it/ppl/Download/}.
 The @option{--with-ppl} configure option should be used if PPL is not
 installed in your default library search path.
 
-@item CLooG-PPL version 0.15
-
-Necessary to build GCC with the Graphite loop optimizations.  It can
-be downloaded from @uref{ftp://gcc.gnu.org/pub/gcc/infrastructure/}.
-The code in @file{cloog-ppl-0.15.tar.gz} comes from a branch of CLooG
-available from @uref{http://repo.or.cz/w/cloog-ppl.git}.  CLooG-PPL
-should be configured with @option{--with-ppl}.
-
-The @option{--with-cloog} configure option should be used if CLooG is
-not installed in your default library search path.
+@item CLooG-PPL version 0.15 / CLooG 0.16
+
+Necessary to build GCC with the Graphite loop optimizations.  There
+are two versions available.  CLooG-PPL 0.15 as well as CLooG 0.16.
+The former is the default right now.  It can be downloaded from
+@uref{ftp://gcc.gnu.org/pub/gcc/infrastructure/} as
+@file{cloog-ppl-0.15.tar.gz}.
+
+CLooG 0.16 support is still in testing stage, but will be the
+default in future GCC releases.  It is also available at
+@uref{ftp://gcc.gnu.org/pub/gcc/infrastructure/} as
+@file{cloog-0.16.x.tar.gz}.  To use it add the additional configure
+option @option{--enable-cloog-backend=isl}.  Even if CLooG 0.16
+does not use PPL, PPL is still required for Graphite.
+
+In both cases @option{--with-cloog} configure option should be used
+if CLooG is not installed in your default library search path.
 
 @item @command{jar}, or InfoZIP (@command{zip} and @command{unzip})