[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