[cfe-commits] [patch][libcxx] Remove some apparently unnecessary and leaked objects from cmp_nullptr tests

Andrew C. Morrow andrew.c.morrow at gmail.com
Thu Aug 2 11:00:12 PDT 2012


Hi -

There are two tests under test/utilities/memory that heap allocate two
integers which remain unused and are subsequently leaked, so the test
fail when run under valgrind. Unless I'm overlooking a subtle reason
why they are needed I think they can be removed, allowing these tests
to pass under valgrind. The attached patch removes the variables. If
there is a reason for them to exist, I can change this to just delete
them at the end of the test.

Thanks,
Andrew
-------------- next part --------------
A non-text attachment was scrubbed...
Name: libcxx.cmp_nullptr.leak.patch
Type: application/octet-stream
Size: 1258 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120802/1c1a9107/attachment.obj>


More information about the cfe-commits mailing list