Re: [PATCH] IPA ICF: memory leak fix

2014-11-27 Thread Richard Biener
On Thu, Nov 27, 2014 at 3:36 PM, Martin Liška wrote: > On 11/23/2014 10:01 AM, Markus Trippelsdorf wrote: >> >> On 2014.11.22 at 17:46 +0100, Markus Trippelsdorf wrote: >>> >>> On 2014.11.22 at 16:04 +0100, Martin Liška wrote: On 11/22/2014 10:09 AM, Markus Trippelsdorf wrote: >

Re: [PATCH] IPA ICF: memory leak fix

2014-11-27 Thread Martin Liška
On 11/23/2014 10:01 AM, Markus Trippelsdorf wrote: On 2014.11.22 at 17:46 +0100, Markus Trippelsdorf wrote: On 2014.11.22 at 16:04 +0100, Martin Liška wrote: On 11/22/2014 10:09 AM, Markus Trippelsdorf wrote: On 2014.11.22 at 09:05 +0100, Martin Liška wrote: Following patch removes memory le

Re: [PATCH] IPA ICF: memory leak fix

2014-11-23 Thread Markus Trippelsdorf
On 2014.11.22 at 17:46 +0100, Markus Trippelsdorf wrote: > On 2014.11.22 at 16:04 +0100, Martin Liška wrote: > > On 11/22/2014 10:09 AM, Markus Trippelsdorf wrote: > > > On 2014.11.22 at 09:05 +0100, Martin Liška wrote: > > >> > > >> Following patch removes memory leak that was introduced by very f

Re: [PATCH] IPA ICF: memory leak fix

2014-11-22 Thread Markus Trippelsdorf
On 2014.11.22 at 16:04 +0100, Martin Liška wrote: > On 11/22/2014 10:09 AM, Markus Trippelsdorf wrote: > > On 2014.11.22 at 09:05 +0100, Martin Liška wrote: > >> > >> Following patch removes memory leak that was introduced by very first IPA > >> ICF patch. > >> I would like to thank David for hunt

Re: [PATCH] IPA ICF: memory leak fix

2014-11-22 Thread Martin Liška
Hello. Thank you for testing, problem is that I should grow the vector by 1, because '0' is used as NULL value. Please try my fixed patch. Thanks, Martin >From 7280e2c8de246c72d2608b5c58590f4fabaf6234 Mon Sep 17 00:00:00 2001 From: mliska Date: Fri, 21 Nov 2014 16:04:06 +0100

Re: [PATCH] IPA ICF: memory leak fix

2014-11-22 Thread Markus Trippelsdorf
On 2014.11.22 at 09:05 +0100, Martin Liška wrote: > Hello. > > Following patch removes memory leak that was introduced by very first IPA ICF > patch. > I would like to thank David for hunting the leak. > > Patch an bootstrap on x86_86-linux-pc and no regression is introduced. I gave the patch a

[PATCH] IPA ICF: memory leak fix

2014-11-22 Thread Martin Liška
:00 2001 From: mliska Date: Fri, 21 Nov 2014 16:04:06 +0100 Subject: [PATCH] IPA ICF: memory leak fix gcc/ChangeLog: 2014-11-21 David Malcolm Martin Liska * ipa-icf.c (sem_function::equals_private): auto_vec replaces int* allocated with XNEWVEC. (sem_function::bb_dict_test): Likew