Re: [PATCH] Fix some leaks and one uninitialized var read

2012-08-22 Thread Steven Bosscher
On Tue, Aug 21, 2012 at 10:33 PM, Jakub Jelinek wrote: > Hi! > > The recent change in find_assert_locations from XCNEWVEC to XNEWVEC > caused a valgrind warning, because bb_rpo[ENTRY_BLOCK] used to > be accessed, but was never initialized. Sorry for this breakage. I looked at each of the changes

Re: [PATCH] Fix some leaks and one uninitialized var read

2012-08-22 Thread Richard Guenther
On Tue, Aug 21, 2012 at 10:33 PM, Jakub Jelinek wrote: > Hi! > > The recent change in find_assert_locations from XCNEWVEC to XNEWVEC > caused a valgrind warning, because bb_rpo[ENTRY_BLOCK] used to > be accessed, but was never initialized. > > Fixed by ignoring edges from ENTRY_BLOCK altogether. >

[PATCH] Fix some leaks and one uninitialized var read

2012-08-21 Thread Jakub Jelinek
Hi! The recent change in find_assert_locations from XCNEWVEC to XNEWVEC caused a valgrind warning, because bb_rpo[ENTRY_BLOCK] used to be accessed, but was never initialized. Fixed by ignoring edges from ENTRY_BLOCK altogether. The rest are a couple of memory leak fixes. Bootstrapped/regtested