This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH 7/N] Fix newly introduced memory leak in tree-ssa-loop-ivopts.c
- From: "Bin.Cheng" <amker dot cheng at gmail dot com>
- To: Martin LiÅka <mliska at suse dot cz>
- Cc: gcc-patches List <gcc-patches at gcc dot gnu dot org>, Bin Cheng <bin dot cheng at arm dot com>
- Date: Fri, 27 Nov 2015 11:54:08 +0800
- Subject: Re: [PATCH 7/N] Fix newly introduced memory leak in tree-ssa-loop-ivopts.c
- Authentication-results: sourceware.org; auth=none
- References: <cover dot 1448286565 dot git dot mliska at suse dot cz> <565774DC dot 5020808 at suse dot cz>
On Fri, Nov 27, 2015 at 5:08 AM, Martin LiÅka <mliska@suse.cz> wrote:
> Hi.
>
> There's one more patch that fixes really of lot memory leaks related to loop
> ivopts.
> The regression was introduced by r230647.
>
> Patch was tested in the series with the rest and the compiler bootstraps
> successfully.
>
> Ready for trunk?
Hi Martin,
Thanks for fixing my issue. The IVO part of patch is OK.
Just for me to understand, iv_common_cand is freed via free_ptr_hash,
and thus typed_free_remove. So what leaks is the iv_use * vector in
struct iv_common_cand, right? I did forget to free that.
BTW, how do you monitor memory use in GCC, maybe I can run same test
for my future patches.
Thanks,
bin