Re: [PATCH] Fix memory leak in IPA passes

2015-04-14 Thread Jan Hubicka
> > OK, I'm going to commit it to trunk. Thanks, and forgot to comment. I do not consider this critical for 5.1. We may want to backport for 5.2. Honza

Re: [PATCH] Fix memory leak in IPA passes

2015-04-14 Thread Martin Liška
On 04/14/2015 03:45 PM, Jan Hubicka wrote: 2015-04-13 Martin Liska * ipa-cp.c (ipcp_driver): Release prev_edge_clone. * ipa-icf.c (sem_item_optimizer::subdivide_classes_by_sensitive_refs): Release symbol_compare_collection. * ipa-reference.c: Add TODO that a v

Re: [PATCH] Fix memory leak in IPA passes

2015-04-14 Thread Jan Hubicka
> > 2015-04-13 Martin Liska > > * ipa-cp.c (ipcp_driver): Release prev_edge_clone. > * ipa-icf.c (sem_item_optimizer::subdivide_classes_by_sensitive_refs): > Release symbol_compare_collection. > * ipa-reference.c: Add TODO that a vector should be released. > --- > gcc/

Re: [PATCH] Fix memory leak in IPA passes

2015-04-14 Thread Martin Liška
On 04/14/2015 10:18 AM, Jakub Jelinek wrote: On Tue, Apr 14, 2015 at 10:14:06AM +0200, Martin Liška wrote: As originally reported by Andi Kleen, following patch fix memory leaks that can be seen in IPA ICF and IPA CP. Patch can bootstrap on x86_64-linux-gnu and survives regression tests. Moreov

Re: [PATCH] Fix memory leak in IPA passes

2015-04-14 Thread Jakub Jelinek
On Tue, Apr 14, 2015 at 10:14:06AM +0200, Martin Liška wrote: > As originally reported by Andi Kleen, following patch fix memory leaks that > can > be seen in IPA ICF and IPA CP. > > Patch can bootstrap on x86_64-linux-gnu and survives regression tests. > Moreover, with patch applied, we can buil

[PATCH] Fix memory leak in IPA passes

2015-04-14 Thread Martin Liška
Hello. As originally reported by Andi Kleen, following patch fix memory leaks that can be seen in IPA ICF and IPA CP. Patch can bootstrap on x86_64-linux-gnu and survives regression tests. Moreover, with patch applied, we can build Firefox and Linux kernel on ppc64le-linux-gnu. Ready for both