Re: [PATCH] Fix PR65538

2015-03-25 Thread Martin Liška
On 03/25/2015 12:37 AM, Jan Hubicka wrote: On Tue, Mar 24, 2015 at 10:54:25PM +0100, Martin Liška wrote: --- a/gcc/symbol-summary.h +++ b/gcc/symbol-summary.h @@ -81,6 +81,12 @@ public: m_symtab_insertion_hook = NULL; m_symtab_removal_hook = NULL; m_symtab_duplication_hook = NU

Re: [PATCH] Fix PR65538

2015-03-24 Thread Jan Hubicka
> On Tue, Mar 24, 2015 at 10:54:25PM +0100, Martin Liška wrote: > > --- a/gcc/symbol-summary.h > > +++ b/gcc/symbol-summary.h > > @@ -81,6 +81,12 @@ public: > > m_symtab_insertion_hook = NULL; > > m_symtab_removal_hook = NULL; > > m_symtab_duplication_hook = NULL; > > + > > +/* R

Re: [PATCH] Fix PR65538

2015-03-24 Thread Jakub Jelinek
On Tue, Mar 24, 2015 at 10:54:25PM +0100, Martin Liška wrote: > --- a/gcc/symbol-summary.h > +++ b/gcc/symbol-summary.h > @@ -81,6 +81,12 @@ public: > m_symtab_insertion_hook = NULL; > m_symtab_removal_hook = NULL; > m_symtab_duplication_hook = NULL; > + > +/* Release all summari

Re: [PATCH] Fix PR65538

2015-03-24 Thread Martin Liška
On 03/24/2015 06:38 PM, Jan Hubicka wrote: >> Hi. >> >> In following patch, I've added missing delete call for all item summaries >> that are >> allocated within a function_summary container in case the container does not >> use >> GGC memory allocation. >> >> Can boostrap on ppc64le and no regre

Re: [PATCH] Fix PR65538

2015-03-24 Thread Jan Hubicka
> Hi. > > In following patch, I've added missing delete call for all item summaries > that are > allocated within a function_summary container in case the container does not > use > GGC memory allocation. > > Can boostrap on ppc64le and no regression is seen on x86_64-linux-pc. > > Ready for t