[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