http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45159
Thomas Koenig changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45159
--- Comment #28 from Thomas Koenig 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 this one is correct in all ca
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45159
--- Comment #27 from Thomas Koenig 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 two gfc_expr() for easy cas
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45159
--- Comment #26 from Thomas Koenig 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 envelope again? I tried t
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45159
--- Comment #25 from Dominique d'Humieres
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)
1
Warning: Cr
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45159
Mikael Morin changed:
What|Removed |Added
CC||mikael at gcc dot gnu.org
--- Comment #24
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45159
--- Comment #23 from Thomas Koenig 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=gcc&view=rev&rev=167413
Log:
2010-12-02 Thomas Koenig
PR fortran/4515
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45159
--- Comment #22 from Thomas Koenig 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 improvements.
--- 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=gcc&view=rev&rev=163834
Log:
2010-09-03 Thomas Koenig
PR fortran/45159
--- 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=gcc&view=rev&rev=163584
Log:
2010-08-27 Thomas Koenig
PR fortran/45159
--- 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 recentl
--- 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 #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 #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 #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:
a(10:1
--- 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=gcc&view=rev&rev=162966
Log:
2010-08-06 Thomas Koenig
PR fortran/45159
--- 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=gcc&view=rev&rev=162848
Log:
2010-08-03 Thomas Koenig
PR fortran/45159
--- 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 ( i>i1 ) x(i:i+nxy-1) = x(i:i+nxy-1) - au3(i-nxy:i-1)*x(i-nxy:i-1)
> 1
> Warn
--- 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=gcc&view=rev&rev=162829
Log:
2010-08-02 Thomas Koenig
PR fortran/45159
--- 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
===
--- dep
--- 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 = transpose
--- 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 re
22 matches
Mail list logo
