[PATCH] [libcxx] Consolidate new/delete replacement in tests and disable it when using sanitizers.
Eric Fiselier
eric at efcs.ca
Tue Dec 9 14:43:17 PST 2014
================
Comment at: test/utilities/memory/default.allocator/allocator.members/allocate.pass.cpp:27
@@ -26,3 @@
- ++new_called;
- assert(s == 3 * sizeof(int));
- return std::malloc(s);
----------------
jroelofs wrote:
> I think we're losing test coverage now that this assert is gone :(
We shouldn't be. I added assertions on line 38 and line 47 that check the same thing.
http://reviews.llvm.org/D6562
More information about the cfe-commits
mailing list