This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch, fortran] PR4773 - [4.6 Regression] Unnecessary temporaries increase the runtime for channel.f90 by ~70%
- From: Paul Richard Thomas <paul dot richard dot thomas at gmail dot com>
- To: Tobias Burnus <burnus at net-b dot de>
- Cc: fortran at gcc dot gnu dot org, gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 12 Jul 2010 14:23:49 +0200
- Subject: Re: [Patch, fortran] PR4773 - [4.6 Regression] Unnecessary temporaries increase the runtime for channel.f90 by ~70%
- References: <AANLkTikanMhKUaI75kr6-BjdMSaFwLXNVQK1-2dRIzbK@mail.gmail.com> <4C3AC918.5080701@net-b.de>
Dear Tobias,
> I was wondering whether one should add a test case (-Warray-temporaries).
That's not a bad idea. I was very reluctant to add any more testcases
that test the tree-dump-original output. They tend to be a complete
pain in the behind from the maintenance point of view. Will do!
>
> OK for the trunk - and for the branches, for which the array-temporary
> fix has been applied to.
I have gone back to 4.4. In the latter, I applied the array-temporary
fix as well. For some reason, applying the overall patch to 4.3
throws up a number of very strange looking regressions. I am inclined
to close the book on this if I cannot see a quick fix tonight. As I
say elsewhere, the underlying bug has been present in gfortran for
about ten years and it has only emerged now!
Thanks
Paul