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

Galina Kistanova gkistanova at gmail.com
Thu May 6 14:33:44 PDT 2010


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
>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: patch01.diff
Type: application/octet-stream
Size: 1928 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20100506/f86867af/attachment.obj>


More information about the llvm-commits mailing list