http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45159
Thomas Koenig tkoenig at gcc dot gnu.org changed:
What|Removed |Added
Status|ASSIGNED
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45159
--- Comment #28 from Thomas Koenig tkoenig at gcc dot gnu.org 2013-02-02
21:31:37 UTC ---
Created attachment 29340
-- http://gcc.gnu.org/bugzilla/attachment.cgi?id=29340
patch which implements comment #27
Still have to verify that
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45159
--- Comment #27 from Thomas Koenig tkoenig at gcc dot gnu.org 2013-02-01
18:16:30 UTC ---
To allow expressions like
a(n:2*n:2) = a(n+1:2*n+1:2)
to be optimized, I will try to write a function which calculates the difference
between
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45159
--- Comment #26 from Thomas Koenig tkoenig at gcc dot gnu.org 2012-06-30
17:20:10 UTC ---
(In reply to comment #19)
Hi Dominique,
(I have done the proof on the back of an envelope that I cannot find right
now.)
Can you find that particular
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45159
Mikael Morin mikael at gcc dot gnu.org changed:
What|Removed |Added
CC||mikael at gcc dot
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45159
--- Comment #25 from Dominique d'Humieres dominiq at lps dot ens.fr
2012-06-29 21:44:56 UTC ---
Anything left to be done?
I see
[macbook] f90/bug% gfc -Warray-temporaries pr45159_4_red.f90
pr45159_4_red.f90:7.15:
a(-3:9:3) = a(-6:18:6)
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45159
--- Comment #23 from Thomas Koenig tkoenig at gcc dot gnu.org 2010-12-03
10:35:16 UTC ---
Author: tkoenig
Date: Fri Dec 3 10:35:12 2010
New Revision: 167413
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=167413
Log:
2010-12-02 Thomas
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45159
--- Comment #22 from Thomas Koenig tkoenig at gcc dot gnu.org 2010-11-27
13:16:54 UTC ---
Created attachment 22546
-- http://gcc.gnu.org/bugzilla/attachment.cgi?id=22546
Patch for some more improvements
Here's a patch for some more
--- Comment #21 from tkoenig at gcc dot gnu dot org 2010-09-03 16:17
---
Subject: Bug 45159
Author: tkoenig
Date: Fri Sep 3 16:16:34 2010
New Revision: 163834
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163834
Log:
2010-09-03 Thomas Koenig tkoe...@gcc.gnu.org
PR
--- Comment #20 from tkoenig at gcc dot gnu dot org 2010-08-27 12:09
---
Subject: Bug 45159
Author: tkoenig
Date: Fri Aug 27 12:08:47 2010
New Revision: 163584
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163584
Log:
2010-08-27 Thomas Koenig tkoe...@gcc.gnu.org
PR
--- Comment #17 from dominiq at lps dot ens dot fr 2010-08-10 08:45 ---
With the patch in comment#16, there is no temporary created for the code in
comment #15, but one is created for
a(10:16:1) = a(11:17)
This seems to be fixed if I replace
+ if (r_stride)
--- Comment #18 from tkoenig at netcologne dot de 2010-08-10 09:19 ---
Subject: Re: Unnecessary temporaries
Am Dienstag, den 10.08.2010, 08:45 + schrieb dominiq at lps dot ens
dot fr:
I think that
+ identical_strides = gfc_dep_compare_expr (l_stride, r_stride)
== 1;
--- Comment #19 from dominiq at lps dot ens dot fr 2010-08-10 12:00 ---
(In reply to comment #18)
Although it is probably better set the stride during resolution.
Probably.
A few comments before leaving for ten days without access to the net.
(1) I think all the changes done
--- Comment #15 from tkoenig at gcc dot gnu dot org 2010-08-09 21:54
---
Here's another case where we generate a temporary:
program main
integer a(100)
a(10:16) = a(11:17:1)
end program main
i...@linux-fd1f:~/Krempel/Dep-9 gfortran -Warray-temporaries d1.f90
d1.f90:3.13:
--- Comment #16 from tkoenig at gcc dot gnu dot org 2010-08-09 22:56
---
(In reply to comment #15)
Here's another case where we generate a temporary:
program main
integer a(100)
a(10:16) = a(11:17:1)
end program main
Here's a tentative patch:
Index: dependency.c
--- Comment #14 from tkoenig at gcc dot gnu dot org 2010-08-06 22:33
---
Subject: Bug 45159
Author: tkoenig
Date: Fri Aug 6 22:33:37 2010
New Revision: 162966
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=162966
Log:
2010-08-06 Thomas Koenig tkoe...@gcc.gnu.org
PR
--- Comment #13 from tkoenig at gcc dot gnu dot org 2010-08-03 22:02
---
Subject: Bug 45159
Author: tkoenig
Date: Tue Aug 3 22:02:30 2010
New Revision: 162848
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=162848
Log:
2010-08-03 Thomas Koenig tkoe...@gcc.gnu.org
PR
--- Comment #8 from burnus at gcc dot gnu dot org 2010-08-02 14:10 ---
(In reply to comment #7)
I fail to see why a scalar temporary is not enough:
Well, try:
I concede defeat.
If one want to enter this kind of reduced temporaries, another candidate is
a=transpose(a) which
--- Comment #9 from dominiq at lps dot ens dot fr 2010-08-02 14:36 ---
Well, with an array descriptor it one only changes the strides/bounds ...
From polyhedron test induct.f90
[macbook] lin/test% gfc -Warray-temporaries induct.f90
induct.f90:1629.20:
rotate_quad =
--- Comment #10 from tkoenig at gcc dot gnu dot org 2010-08-02 19:00
---
This fixes the test case from comment #0, and looks much more sane.
Let's see if this survives regression-testing...
Index: dependency.c
===
---
--- Comment #11 from tkoenig at gcc dot gnu dot org 2010-08-02 22:04
---
Subject: Bug 45159
Author: tkoenig
Date: Mon Aug 2 22:04:36 2010
New Revision: 162829
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=162829
Log:
2010-08-02 Thomas Koenig tkoe...@gcc.gnu.org
PR
--- Comment #12 from tkoenig at gcc dot gnu dot org 2010-08-02 22:16
---
(In reply to comment #5)
[macbook] lin/test% gfc -Warray-temporaries nf.f90
nf.f90:293.30:
if ( ii1 ) x(i:i+nxy-1) = x(i:i+nxy-1) - au3(i-nxy:i-1)*x(i-nxy:i-1)
1
Warning:
22 matches
Mail list logo
