This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch, fortran] PR67177, 67977 and memory leaks in move_alloc
- From: Paul Richard Thomas <paul dot richard dot thomas at gmail dot com>
- To: Steve Kargl <sgk at troutmask dot apl dot washington dot edu>
- Cc: "fortran at gcc dot gnu dot org" <fortran at gcc dot gnu dot org>, gcc-patches <gcc-patches at gcc dot gnu dot org>, Dominique Dhumieres <dominiq at lps dot ens dot fr>
- Date: Sun, 18 Oct 2015 11:34:13 +0200
- Subject: Re: [Patch, fortran] PR67177, 67977 and memory leaks in move_alloc
- Authentication-results: sourceware.org; auth=none
- References: <CAGkQGiLNMzhCh7gZP6yDjCQbuMy1YX-uWwSzg_mP4Y83XrP6gg at mail dot gmail dot com> <20151017205133 dot GA86576 at troutmask dot apl dot washington dot edu>
Hi Steve,
Thanks - Committed as revision 228940.
I'll commit to 5 branch next Sunday.
Cheers
Paul
On 17 October 2015 at 22:51, Steve Kargl
<sgk@troutmask.apl.washington.edu> wrote:
> On Sat, Oct 17, 2015 at 09:49:45PM +0200, Paul Richard Thomas wrote:
>>
>> I was moved by a report on clf of memory leaks in move_alloc to
>> investigate the cause. This turned out to be trivial but led to the
>> above PRs, which themselves were trivial. The result is the attached
>> patch. I am aware that I have not investigated the further
>> ramifications that I can imagine are there. Rather, I thought just to
>> fix the reported problems.
>>
>> It should be noted that there is no PR directly associated with the
>> memory leaks. Since the standard does not require this, I did not
>> think that it was worthwhile to raise a PR and then close it!
>>
>> Bootstraps and regtests on FC21/x86_64 - OK for trunk? ... and 5.2
>> after a decent interval?
>
> OK.
>
> --
> steve
--
Outside of a dog, a book is a man's best friend. Inside of a dog it's
too dark to read.
Groucho Marx