* [patch, fortran] Eliminate duplicate function calls with rank>0 and unknown shape
@ 2011-05-01 9:50 Thomas Koenig
2011-05-03 20:20 ` Thomas Koenig
2011-05-14 14:21 ` Jerry DeLisle
0 siblings, 2 replies; 6+ messages in thread
From: Thomas Koenig @ 2011-05-01 9:50 UTC (permalink / raw)
To: fortran, gcc-patches
[-- Attachment #1: Type: text/plain, Size: 1140 bytes --]
Hello world,
after Paul's fix for allocate on assignment (thanks Paul!), here is a
patch for the original test case from PR 22572, where the bounds of
the function are unknown at compile time. This uses an allocatable
temporary.
In the long run, another option is to use interface mapping to evaluate
the bounds of intrinsics and explicit-shape functions. For this, it
would be necessary to write a front-end-only version of
gfc_evaluate_now, which would be complicated by the desire not to
disturb common function elimination, so I've put that on the back burner
for now.
Regression-tested. OK for trunk?
Thomas
2011-05-01 Thomas Koenig <tkoenig@gcc.gnu.org>
PR fortran/22572
* frontend-passes.c (cfe_register_funcs): Also register functions
for potential elimination if the rank is > 0, the shape is unknown
and reallocate on assignment is active.
(create_var): For rank > 0 functions with unknown shape, create
an allocatable temporary.
2011-05-01 Thomas Koenig <tkoenig@gcc.gnu.org>
PR fortran/22572
* function_optimize_7.f90: New test case.
[-- Attachment #2: realloc.diff --]
[-- Type: text/x-patch, Size: 2389 bytes --]
Index: frontend-passes.c
===================================================================
--- frontend-passes.c (Revision 173214)
+++ frontend-passes.c (Arbeitskopie)
@@ -152,11 +152,11 @@ cfe_register_funcs (gfc_expr **e, int *walk_subtre
if ((*e)->ts.type == BT_CHARACTER)
return 0;
- /* If we don't know the shape at compile time, we do not create a temporary
- variable to hold the intermediate result. FIXME: Change this later when
- allocation on assignment works for intrinsics. */
+ /* If we don't know the shape at compile time, we create an allocatable
+ temporary variable to hold the intermediate result, but only if
+ allocation on assignment is active. */
- if ((*e)->rank > 0 && (*e)->shape == NULL)
+ if ((*e)->rank > 0 && (*e)->shape == NULL && !gfc_option.flag_realloc_lhs)
return 0;
/* Skip the test for pure functions if -faggressive-function-elimination
@@ -250,22 +250,38 @@ create_var (gfc_expr * e)
symbol = symtree->n.sym;
symbol->ts = e->ts;
- symbol->as = gfc_get_array_spec ();
- symbol->as->rank = e->rank;
- symbol->as->type = AS_EXPLICIT;
- for (i=0; i<e->rank; i++)
+
+ if (e->rank > 0)
{
- gfc_expr *p, *q;
+ symbol->as = gfc_get_array_spec ();
+ symbol->as->rank = e->rank;
+
+ if (e->shape == NULL)
+ {
+ /* We don't know the shape at compile time, so we use an
+ allocatable. */
+ symbol->as->type = AS_DEFERRED;
+ symbol->attr.allocatable = 1;
+ }
+ else
+ {
+ symbol->as->type = AS_EXPLICIT;
+ /* Copy the shape. */
+ for (i=0; i<e->rank; i++)
+ {
+ gfc_expr *p, *q;
- p = gfc_get_constant_expr (BT_INTEGER, gfc_default_integer_kind,
- &(e->where));
- mpz_set_si (p->value.integer, 1);
- symbol->as->lower[i] = p;
-
- q = gfc_get_constant_expr (BT_INTEGER, gfc_index_integer_kind,
- &(e->where));
- mpz_set (q->value.integer, e->shape[i]);
- symbol->as->upper[i] = q;
+ p = gfc_get_constant_expr (BT_INTEGER, gfc_default_integer_kind,
+ &(e->where));
+ mpz_set_si (p->value.integer, 1);
+ symbol->as->lower[i] = p;
+
+ q = gfc_get_constant_expr (BT_INTEGER, gfc_index_integer_kind,
+ &(e->where));
+ mpz_set (q->value.integer, e->shape[i]);
+ symbol->as->upper[i] = q;
+ }
+ }
}
symbol->attr.flavor = FL_VARIABLE;
[-- Attachment #3: function_optimize_7.f90 --]
[-- Type: text/plain, Size: 1796 bytes --]
! { dg-do compile }
! { dg-options "-O -fdump-tree-original -Warray-temporaries" }
subroutine xx(n, m, a, b, c, d, x, z, i, s_in, s_out)
implicit none
integer, intent(in) :: n, m
real, intent(in), dimension(n,n) :: a, b, c
real, intent(out), dimension(n,n) :: d
real, intent(in), dimension(n,m) :: s_in
real, intent(out), dimension(m) :: s_out
integer, intent(out) :: i
real, intent(inout) :: x
real, intent(out) :: z
character(60) :: line
real, external :: ext_func
interface
elemental function element(x)
real, intent(in) :: x
real :: elem
end function element
pure function mypure(x)
real, intent(in) :: x
integer :: mypure
end function mypure
elemental impure function elem_impure(x)
real, intent(in) :: x
real :: elem_impure
end function elem_impure
end interface
d = matmul(a,b) + matmul(a,b) ! { dg-warning "Creating array temporary" }
z = sin(x) + cos(x) + sin(x) + cos(x)
x = ext_func(a) + 23 + ext_func(a)
z = element(x) + element(x)
i = mypure(x) - mypure(x)
z = elem_impure(x) - elem_impure(x)
s_out = sum(s_in,1) + 3.14 / sum(s_in,1) ! { dg-warning "Creating array temporary" }
end subroutine xx
! { dg-final { scan-tree-dump-times "matmul_r4" 1 "original" } }
! { dg-final { scan-tree-dump-times "__builtin_sinf" 1 "original" } }
! { dg-final { scan-tree-dump-times "__builtin_cosf" 1 "original" } }
! { dg-final { scan-tree-dump-times "ext_func" 2 "original" } }
! { dg-final { scan-tree-dump-times "element" 1 "original" } }
! { dg-final { scan-tree-dump-times "mypure" 1 "original" } }
! { dg-final { scan-tree-dump-times "elem_impure" 2 "original" } }
! { dg-final { scan-tree-dump-times "sum_r4" 1 "original" } }
! { dg-final { cleanup-tree-dump "original" } }
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [patch, fortran] Eliminate duplicate function calls with rank>0 and unknown shape
2011-05-01 9:50 [patch, fortran] Eliminate duplicate function calls with rank>0 and unknown shape Thomas Koenig
@ 2011-05-03 20:20 ` Thomas Koenig
2011-05-14 0:45 ` Thomas Koenig
2011-05-14 14:21 ` Jerry DeLisle
1 sibling, 1 reply; 6+ messages in thread
From: Thomas Koenig @ 2011-05-03 20:20 UTC (permalink / raw)
To: fortran, gcc-patches
Am 01.05.2011 11:49, schrieb Thomas Koenig:
> Hello world,
>
> after Paul's fix for allocate on assignment (thanks Paul!), here is a
> patch for the original test case from PR 22572, where the bounds of
> the function are unknown at compile time. This uses an allocatable
> temporary.
Ping?
Thomas
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [patch, fortran] Eliminate duplicate function calls with rank>0 and unknown shape
2011-05-03 20:20 ` Thomas Koenig
@ 2011-05-14 0:45 ` Thomas Koenig
2011-05-14 14:10 ` Steve Kargl
0 siblings, 1 reply; 6+ messages in thread
From: Thomas Koenig @ 2011-05-14 0:45 UTC (permalink / raw)
To: fortran, gcc-patches
Am 03.05.2011 22:12, schrieb Thomas Koenig:
> Am 01.05.2011 11:49, schrieb Thomas Koenig:
>> Hello world,
>>
>> after Paul's fix for allocate on assignment (thanks Paul!), here is a
>> patch for the original test case from PR 22572, where the bounds of
>> the function are unknown at compile time. This uses an allocatable
>> temporary.
>
> Ping?
Ping**2?
There's been a bit of offline discussion with Tobias regarding -Os and
timings. At least for a simple test case, the patch made the resulting
program shorter. As for timing, I would confidently expect that, even
with -fstack-arrays, any useful function would take longer than a
malloc()/free() pair.
Thomas
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [patch, fortran] Eliminate duplicate function calls with rank>0 and unknown shape
2011-05-14 0:45 ` Thomas Koenig
@ 2011-05-14 14:10 ` Steve Kargl
0 siblings, 0 replies; 6+ messages in thread
From: Steve Kargl @ 2011-05-14 14:10 UTC (permalink / raw)
To: Thomas Koenig; +Cc: fortran, gcc-patches
On Fri, May 13, 2011 at 10:18:57PM +0200, Thomas Koenig wrote:
> Am 03.05.2011 22:12, schrieb Thomas Koenig:
> >Am 01.05.2011 11:49, schrieb Thomas Koenig:
> >>Hello world,
> >>
> >>after Paul's fix for allocate on assignment (thanks Paul!), here is a
> >>patch for the original test case from PR 22572, where the bounds of
> >>the function are unknown at compile time. This uses an allocatable
> >>temporary.
> >
> >Ping?
>
> Ping**2?
>
> There's been a bit of offline discussion with Tobias regarding -Os and
> timings. At least for a simple test case, the patch made the resulting
> program shorter. As for timing, I would confidently expect that, even
> with -fstack-arrays, any useful function would take longer than a
> malloc()/free() pair.
>
> Thomas
Thomas, I can't find the diff. I read it over if you give me a pointer.
--
Steve
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [patch, fortran] Eliminate duplicate function calls with rank>0 and unknown shape
2011-05-01 9:50 [patch, fortran] Eliminate duplicate function calls with rank>0 and unknown shape Thomas Koenig
2011-05-03 20:20 ` Thomas Koenig
@ 2011-05-14 14:21 ` Jerry DeLisle
2011-05-14 14:56 ` Thomas Koenig
1 sibling, 1 reply; 6+ messages in thread
From: Jerry DeLisle @ 2011-05-14 14:21 UTC (permalink / raw)
To: Thomas Koenig; +Cc: fortran, gcc-patches
On 05/01/2011 02:49 AM, Thomas Koenig wrote:
> Hello world,
>
> after Paul's fix for allocate on assignment (thanks Paul!), here is a
> patch for the original test case from PR 22572, where the bounds of
> the function are unknown at compile time. This uses an allocatable
> temporary.
>
> In the long run, another option is to use interface mapping to evaluate
> the bounds of intrinsics and explicit-shape functions. For this, it
> would be necessary to write a front-end-only version of gfc_evaluate_now, which
> would be complicated by the desire not to disturb common function elimination,
> so I've put that on the back burner for now.
>
> Regression-tested. OK for trunk?
>
> Thomas
OK and thanks for patch.
Jerry
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [patch, fortran] Eliminate duplicate function calls with rank>0 and unknown shape
2011-05-14 14:21 ` Jerry DeLisle
@ 2011-05-14 14:56 ` Thomas Koenig
0 siblings, 0 replies; 6+ messages in thread
From: Thomas Koenig @ 2011-05-14 14:56 UTC (permalink / raw)
To: Jerry DeLisle; +Cc: fortran, gcc-patches
Hi Jerry,
> On 05/01/2011 02:49 AM, Thomas Koenig wrote:
>> Hello world,
>>
>> after Paul's fix for allocate on assignment (thanks Paul!), here is a
>> patch for the original test case from PR 22572, where the bounds of
>> the function are unknown at compile time. This uses an allocatable
>> temporary.
>>
>> In the long run, another option is to use interface mapping to evaluate
>> the bounds of intrinsics and explicit-shape functions. For this, it
>> would be necessary to write a front-end-only version of
>> gfc_evaluate_now, which
>> would be complicated by the desire not to disturb common function
>> elimination,
>> so I've put that on the back burner for now.
>>
>> Regression-tested. OK for trunk?
>>
>> Thomas
> OK and thanks for patch.
>
Waiting for Emacs...
Sende fortran/ChangeLog
Sende fortran/frontend-passes.c
Sende testsuite/ChangeLog
HinzufÃŒgen testsuite/gfortran.dg/function_optimize_7.f90
Ãbertrage Daten ....
Revision 173752 ÃŒbertragen.
Thanks for the review!
I'll close the PR itself and add a new one for not using allocatable arrays.
Thomas
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2011-05-14 9:50 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-05-01 9:50 [patch, fortran] Eliminate duplicate function calls with rank>0 and unknown shape Thomas Koenig
2011-05-03 20:20 ` Thomas Koenig
2011-05-14 0:45 ` Thomas Koenig
2011-05-14 14:10 ` Steve Kargl
2011-05-14 14:21 ` Jerry DeLisle
2011-05-14 14:56 ` Thomas Koenig
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).