Re: [PATCH 10/21] PR jit/63854: Fix leak of worklist within jit-recording.c

2014-11-20 Thread Jeff Law
On 11/20/14 09:01, Richard Biener wrote: Is there a governance distinction here, between patch review vs decisions of the steering committee? i.e. do changes to the maintainers part of the MAINTAINERS file require higher-level approval? Yes, reviewers and maintainers are appointed by the steer

Re: [PATCH 10/21] PR jit/63854: Fix leak of worklist within jit-recording.c

2014-11-20 Thread Richard Biener
On Wed, Nov 19, 2014 at 9:02 PM, David Malcolm wrote: > On Wed, 2014-11-19 at 09:59 -0700, Jeff Law wrote: >> On 11/19/14 03:46, David Malcolm wrote: >> > Fix this leak: >> > >> > 160 bytes in 5 blocks are definitely lost in loss record 154 of 228 >> > at 0x4A0645D: malloc (in >> > /usr/lib64

Re: [PATCH 10/21] PR jit/63854: Fix leak of worklist within jit-recording.c

2014-11-19 Thread David Malcolm
On Wed, 2014-11-19 at 09:59 -0700, Jeff Law wrote: > On 11/19/14 03:46, David Malcolm wrote: > > Fix this leak: > > > > 160 bytes in 5 blocks are definitely lost in loss record 154 of 228 > > at 0x4A0645D: malloc (in > > /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) > > by 0x5D75D

Re: [PATCH 10/21] PR jit/63854: Fix leak of worklist within jit-recording.c

2014-11-19 Thread Jeff Law
On 11/19/14 03:46, David Malcolm wrote: Fix this leak: 160 bytes in 5 blocks are definitely lost in loss record 154 of 228 at 0x4A0645D: malloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) by 0x5D75D4F: xrealloc (xmalloc.c:177) by 0x4DE1710: void va_heap::reserve(vec*

[PATCH 10/21] PR jit/63854: Fix leak of worklist within jit-recording.c

2014-11-19 Thread David Malcolm
Fix this leak: 160 bytes in 5 blocks are definitely lost in loss record 154 of 228 at 0x4A0645D: malloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) by 0x5D75D4F: xrealloc (xmalloc.c:177) by 0x4DE1710: void va_heap::reserve(vec*&, unsigned int, bool) (vec.h:310) by 0x4D