This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch, Fortran] PR fortran/40850 derived type allocatables freed before their components
- From: Mikael Morin <mikael dot morin at sfr dot fr>
- To: fortran at gcc dot gnu dot org
- Cc: Tobias Burnus <tobias dot burnus at physik dot fu-berlin dot de>, gcc-patches at gcc dot gnu dot org
- Date: Wed, 23 Feb 2011 23:50:02 +0100
- Subject: Re: [Patch, Fortran] PR fortran/40850 derived type allocatables freed before their components
- References: <20110223085355.GA9454@physik.fu-berlin.de>
On Wednesday 23 February 2011 09:53:59 Tobias Burnus wrote:
> Mikael Morin wrote:
> > The patch fixes the comment 5 in the PR, but not the rest (comment 8).
> > I will regression test this tomorrow; OK for trunk if it passes?
>
> OK. Thanks for the patch!
>
> > I forgot to add that I would appreciate if the testcase was
> > confirmed to fail without the patch on various platforms
>
> Here, it fails for slighly above 4 kilobyte. I would not make
> the test case allocatate much more data
Ok. I changed the value to 2K (which makes an amount of memory of
2K*sizeof(descriptor))
> - currently its 4 MB -
Well, in fact it was much more (core file was ~50MiB) as it is an array of
descriptors, not of plain integers.
For the record, I narrowed the FreeBSD trigger value between 42K and 43K.
1M was indeed too much.
Thanks for looking.
Committed as revision 170445.
Mikael