This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Memory leak in ifcvt.c
- From: "Richard Guenther" <richard dot guenther at gmail dot com>
- To: "Kenneth Zadeck" <zadeck at naturalbridge dot com>
- Cc: "Daniel Berlin" <dberlin at dberlin dot org>, gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 1 Jul 2008 11:27:10 +0200
- Subject: Re: Memory leak in ifcvt.c
- References: <4aca3dc20806281453wb46eb7bne44c4ed4553e2eeb@mail.gmail.com> <4866CBE8.6080908@naturalbridge.com> <4aca3dc20806281655m3ea840fbwe85d1b9f578ad364@mail.gmail.com> <4868D1A3.6040706@naturalbridge.com> <84fc9c000806300911u12586c62hd8185f996964f4bb@mail.gmail.com> <486934FA.6050502@naturalbridge.com>
On Mon, Jun 30, 2008 at 9:33 PM, Kenneth Zadeck
<zadeck@naturalbridge.com> wrote:
> Updated as richard has requested.
>
> 2008-06-30 Kenneth Zadeck <zadeck@naturalbridge.com>
>
> * df-scan.c (df_scan_free_ref_vec, df_scan_free_mws_vec): New
> macros.
> (df_scan_free_internal): Free data structures not
> allocated in storage pools.
> (df_mw_hardreg_chain_delete_eq_uses): Use df_scan_free_mws_vec.
> (df_refs_add_to_chains): Use df_scan_free_ref_vec and
> df_scan_free_mws_vec.
> * dse.c (dse_step6): Free offset_map_p and offset_map_n
> unconditionally.
>
> 2008-06-30 Kenneth Zadeck <zadeck@naturalbridge.com>
>
> * ifcvt.c (cond_move_process_if_block): Free vectors on false
> return.
>
> Committed as revisions 137284 and 137285.
Thanks. This seems to have a very positive effect on some testcases,
can you back-port these fixes to the 4.3 branch if applicable?
Thanks,
Richard.