[PATCH] D16970: scoped alloc construct: adds basic tests

2017-10-23 Thread Masud Rahman via Phabricator via cfe-commits
frutiger abandoned this revision. frutiger added a comment. @AlisdairM maybe you want to commandeer this revision? I don't think this is going anywhere... https://reviews.llvm.org/D16970 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http

Re: [PATCH] D16970: scoped alloc construct: adds basic tests

2016-02-07 Thread Masud Rahman via cfe-commits
frutiger updated this revision to Diff 47143. frutiger added a comment. Fixes include guards. http://reviews.llvm.org/D16970 Files: test/std/utilities/allocator.adaptor/scoped.adaptor.operators/construct.pass.cpp test/support/instrumentingallocators.h Index: test/support/instrumentingallo

Re: [PATCH] D16970: scoped alloc construct: adds basic tests

2016-02-07 Thread Alisdair Meredith via cfe-commits
AlisdairM added a subscriber: AlisdairM. Comment at: test/support/instrumentingallocators.h:25 @@ +24,3 @@ +~Instrumentation() { +assert(allocs_.empty()); +} Perhaps a loop, showing what leaked? for (auto const& alloc : allocs_) { std::cout <<

[PATCH] D16970: scoped alloc construct: adds basic tests

2016-02-07 Thread Masud Rahman via cfe-commits
frutiger created this revision. frutiger added reviewers: EricWF, mclow.lists. frutiger added a subscriber: cfe-commits. This commit introduces a new supporting allocator 'IA1' in 'support/instrumentingallocators.h' which tracks allocations and deallocations. This commit also introduces a new tes

Re: [PATCH] D16970: scoped alloc construct: adds basic tests

2016-02-07 Thread Masud Rahman via cfe-commits
frutiger updated this revision to Diff 47141. frutiger added a comment. Added `operator==` and `operator!=`. http://reviews.llvm.org/D16970 Files: test/std/utilities/allocator.adaptor/scoped.adaptor.operators/construct.pass.cpp test/support/instrumentingallocators.h Index: test/support/in