[patch, fortran] PR 46065 - optimize trim
Thomas Koenig
tkoenig@netcologne.de
Fri Dec 31 11:53:00 GMT 2010
Am Freitag, den 31.12.2010, 12:13 +0100 schrieb Tobias Burnus:
> Hello Thomas,
>
> Thomas Koenig wrote:
> >> However, I worry that
> >> the following program is mishandled, "sub" is the obvious case while
> >> "two" is a special case, cf. PR 46896.
> > The attached version of the patch fixes your concern by never applying
> > this optimization within an argument list. I have also added your test
> > case.
>
> The patch is OK. One could relax that argument-list restriction and
> allow it for PURE procedures, which is the trivial case - but that can
> (like the ref part) be deferred.
Sende fortran/ChangeLog
Sende fortran/frontend-passes.c
Sende testsuite/ChangeLog
Hinzufügen testsuite/gfortran.dg/trim_optimize_3.f90
Hinzufügen testsuite/gfortran.dg/trim_optimize_4.f90
Ãbertrage Daten .....
Revision 168367 übertragen.
Thanks for the review!
I will update the PR with the comments of what can still be added.
Thomas
More information about the Gcc-patches
mailing list