[PATCH] [libcxxabi] Cleanup memory in tests to placate ASAN.
Eric Fiselier
eric at efcs.ca
Tue Nov 18 16:02:56 PST 2014
================
Comment at: test/test_vector1.cpp:218
@@ -217,3 +219,2 @@
two = __cxxabiv1::__cxa_vec_new2( 10, 40, 8, throw_construct, throw_destruct, my_alloc2, my_dealloc2 );
- three = __cxxabiv1::__cxa_vec_new3( 10, 40, 8, throw_construct, throw_destruct, my_alloc2, my_dealloc3 );
}
----------------
jroelofs wrote:
> I may have missed the conversation about this one, but why are you deleting this part of the test?
>
> I don't fee comfortable signing off on the changes in this file as I'm not familiar with __cxa_vec_new*.
The test seems to check that __cxa_vec_delete deallocates the vector even if one of the destructors throws. The call to `__cxa_vec_delete3` is not called because `__cxa_vec_delete2` is expected to throw. To check this `assert(false)` is now used instead of the side effects of `__cxa_vec_delete3`.
@mclow seems to have written the test so I'll ask him to sign off on it.
http://reviews.llvm.org/D6281
More information about the cfe-commits
mailing list