This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch] plug mem leak in gcse
- From: Richard Guenther <richard dot guenther at gmail dot com>
- To: Steven Bosscher <stevenb dot gcc at gmail dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 28 Mar 2011 15:40:04 +0200
- Subject: Re: [patch] plug mem leak in gcse
- References: <AANLkTi=j=a4RuXQW3wH8EEz=zwUYWY8NWD=gdZUJZNDZ@mail.gmail.com>
On Mon, Mar 28, 2011 at 3:29 PM, Steven Bosscher <stevenb.gcc@gmail.com> wrote:
> Hi,
>
> The bitmap reg_set_bitmap is allocated in alloc_gcse_mem, but it is
> never freed. The foul smell of a mem leak...
>
> Plugged thus. Bootstrapped&tested on x86_64-unknown-linux-gnu. OK for trunk?
Ok.
Thanks,
Richard.
> Ciao!
> Steven
>
>
> * gcse.c (free_gcse_mem): Free reg_set_bitmap.
>
> Index: gcse.c
> ===================================================================
> --- gcse.c ? ? ?(revision 171599)
> +++ gcse.c ? ? ?(working copy)
> @@ -657,6 +657,8 @@
> ?static void
> ?free_gcse_mem (void)
> ?{
> + ?FREE_REG_SET (reg_set_bitmap);
> +
> ? free_modify_mem_tables ();
> ? BITMAP_FREE (modify_mem_list_set);
> ? BITMAP_FREE (blocks_with_calls);
>