[llvm-commits] [PATCH] test-suite: fix memory leak and valloc use

Chris Lattner clattner at apple.com
Thu May 6 14:54:21 PDT 2010


Thanks, applied in r103201

On May 6, 2010, at 2:33 PM, Galina Kistanova wrote:

> Yep. :)  2*N and N actually.
> Thanks for catching this, Chris!
> 
> The changed patch is attached.
> 
> Thanks
> 
> Galina
> 
> 
> On Thu, May 6, 2010 at 2:11 PM, Chris Lattner <clattner at apple.com> wrote:
>> 
>> On May 6, 2010, at 1:48 PM, Galina Kistanova wrote:
>> 
>>> Hello everyone,
>>> 
>>> Please review attached patch. This patch fixes both usage of
>>> non-multiplatform valloc and the memory leak. The only potential issue
>>> is a small static segment.
>> 
>> Shouldn't the array size be 4*N and N for 'w'?
>> 
>> -Chris
>> 
>> 
> <patch01.diff>




More information about the llvm-commits mailing list