[prev in list] [next in list] [prev in thread] [next in thread] 

List:       gcc-fortran
Subject:    Re: [Patch, Fortran] PR fortran/40850 derived type  =?iso-8859-1?q?allocatables=09freed_before_their
From:       Mikael Morin <mikael.morin () sfr ! fr>
Date:       2011-02-23 22:50:02
Message-ID: 201102232350.03078.mikael.morin () sfr ! fr
[Download RAW message or body]

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


[prev in list] [next in list] [prev in thread] [next in thread]