This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch, fortran] PR59016, second version
- From: Tobias Burnus <tobias dot burnus at physik dot fu-berlin dot de>
- To: Mikael Morin <mikael dot morin at sfr dot fr>
- Cc: Evangelos Drikos <drikosev at otenet dot gr>, fortran at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org
- Date: Fri, 10 Apr 2015 12:24:14 +0200
- Subject: Re: [Patch, fortran] PR59016, second version
- Authentication-results: sourceware.org; auth=none
Hello,
Mikael Morin wrote:
> I have finally taken over PR59016.
> The bug was a pointer to a freed (derived type) symbol retained in the
> generic one with the same name, because the symbols weren't versioned.
>
> Evangelos proposed a patch that was freeing the memory by hand. The
> simpler one here, on the other hand, saves the symbol before any
> modification and lets the symbol undo code do its job.
> Thanks to Evangelos anyway for isolating the bug and proposing a fix.
>
> This patch has been tested on x86_64-unknown-linux-gnu.
> OK for trunk/4.9/4.8?
Thanks for the work on this, Evangelos and Mikael!
The patch looks good to me.
Cheers,
Tobias