This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Call symbol_summary<>::release instead of ~symbol_summary (PR, ipa/79285).
- From: Richard Biener <richard dot guenther at gmail dot com>
- To: Martin Liška <mliska at suse dot cz>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Martin Jambor <mjambor at suse dot cz>
- Date: Tue, 31 Jan 2017 10:31:12 +0100
- Subject: Re: [PATCH] Call symbol_summary<>::release instead of ~symbol_summary (PR, ipa/79285).
- Authentication-results: sourceware.org; auth=none
- References: <ee5a67a1-e5fc-6f62-e858-fff41ba4b32f@suse.cz>
On Tue, Jan 31, 2017 at 9:34 AM, Martin Liška <mliska@suse.cz> wrote:
> Hello.
>
> Calling twice ~symbol_summary (once from ipa_free_all_node_params, second time from ggc free page) causes
> double calling ~hash_map, which triggers the valgrind issue. Proper fix is to call ::release from
> ipa_free_all_node_params.
>
> Patch can bootstrap on ppc64le-redhat-linux and survives regression tests.
>
> Ready to be installed?
Ok.
Richard.
> Martin