diff mbox

[05/28] Document availability of cloog-0.16

Message ID 1295638067-13460-6-git-send-email-sebpop@gmail.com
State New
Headers show

Commit Message

Sebastian Pop Jan. 21, 2011, 7:27 p.m. UTC
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(-)

Comments

Gerald Pfeifer Jan. 24, 2011, 12:03 a.m. UTC | #1
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. UTC | #2
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. UTC | #3
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. UTC | #4
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. UTC | #5
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. UTC | #6
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
diff mbox

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})