diff --git a/include/__tree b/include/__tree index 8e5447a2..401891b2 100644 --- a/include/__tree +++ b/include/__tree @@ -522,9 +522,9 @@ public: bool __value_constructed; _LIBCPP_INLINE_VISIBILITY - explicit __tree_node_destructor(allocator_type& __na) _NOEXCEPT + explicit __tree_node_destructor(allocator_type& __na, bool __val = false) _NOEXCEPT : __na_(__na), - __value_constructed(false) + __value_constructed(__val) {} _LIBCPP_INLINE_VISIBILITY @@ -2291,7 +2291,7 @@ __tree<_Tp, _Compare, _Allocator>::remove(const_iterator __p) _NOEXCEPT --size(); __tree_remove(__end_node()->__left_, static_cast<__node_base_pointer>(__np)); - return __node_holder(__np, _Dp(__node_alloc())); + return __node_holder(__np, _Dp(__node_alloc(), true)); } template diff --git a/test/std/containers/associative/map/map.cons/move_alloc.pass.cpp b/test/std/containers/associative/map/map.cons/move_alloc.pass.cpp index 1845e88b..264afcfe 100644 --- a/test/std/containers/associative/map/map.cons/move_alloc.pass.cpp +++ b/test/std/containers/associative/map/map.cons/move_alloc.pass.cpp @@ -20,6 +20,7 @@ #include "../../../test_compare.h" #include "test_allocator.h" #include "min_allocator.h" +#include "Counter.h" int main() { @@ -141,6 +142,53 @@ int main() assert(m3.key_comp() == C(5)); assert(m1.empty()); } + { + typedef Counter T; + typedef std::pair V; + typedef std::pair VC; + typedef test_allocator A; + typedef std::less C; + typedef std::map M; + typedef V* I; + Counter_base::gConstructed = 0; + { + V a1[] = + { + V(1, 1), + V(1, 2), + V(1, 3), + V(2, 1), + V(2, 2), + V(2, 3), + V(3, 1), + V(3, 2), + V(3, 3) + }; + const size_t num = sizeof(a1)/sizeof(a1[0]); + assert(Counter_base::gConstructed == num); + + M m1(I(a1), I(a1+num), C(), A()); + assert(Counter_base::gConstructed == num+3); + + M m2(m1); + assert(m2 == m1); + assert(Counter_base::gConstructed == num+6); + + M m3(std::move(m1), A()); + assert(m3 == m2); + assert(m1.empty()); + assert(Counter_base::gConstructed == num+6); + + { + M m4(std::move(m2), A(5)); + assert(Counter_base::gConstructed == num+6); + assert(m4 == m3); + assert(m2.empty()); + } + assert(Counter_base::gConstructed == num+3); + } + assert(Counter_base::gConstructed == 0); + } #if __cplusplus >= 201103L { typedef std::pair V; diff --git a/test/std/containers/associative/multimap/multimap.cons/move_alloc.pass.cpp b/test/std/containers/associative/multimap/multimap.cons/move_alloc.pass.cpp index 651993a7..75f42bec 100644 --- a/test/std/containers/associative/multimap/multimap.cons/move_alloc.pass.cpp +++ b/test/std/containers/associative/multimap/multimap.cons/move_alloc.pass.cpp @@ -20,6 +20,7 @@ #include "../../../test_compare.h" #include "test_allocator.h" #include "min_allocator.h" +#include "Counter.h" int main() { @@ -141,6 +142,53 @@ int main() assert(m3.key_comp() == C(5)); assert(m1.empty()); } + { + typedef Counter T; + typedef std::pair V; + typedef std::pair VC; + typedef test_allocator A; + typedef std::less C; + typedef std::multimap M; + typedef V* I; + Counter_base::gConstructed = 0; + { + V a1[] = + { + V(1, 1), + V(1, 2), + V(1, 3), + V(2, 1), + V(2, 2), + V(2, 3), + V(3, 1), + V(3, 2), + V(3, 3) + }; + const size_t num = sizeof(a1)/sizeof(a1[0]); + assert(Counter_base::gConstructed == num); + + M m1(I(a1), I(a1+num), C(), A()); + assert(Counter_base::gConstructed == 2*num); + + M m2(m1); + assert(m2 == m1); + assert(Counter_base::gConstructed == 3*num); + + M m3(std::move(m1), A()); + assert(m3 == m2); + assert(m1.empty()); + assert(Counter_base::gConstructed == 3*num); + + { + M m4(std::move(m2), A(5)); + assert(Counter_base::gConstructed == 3*num); + assert(m4 == m3); + assert(m2.empty()); + } + assert(Counter_base::gConstructed == 2*num); + } + assert(Counter_base::gConstructed == 0); + } #if __cplusplus >= 201103L { typedef std::pair V; diff --git a/test/std/containers/associative/multiset/multiset.cons/move_alloc.pass.cpp b/test/std/containers/associative/multiset/multiset.cons/move_alloc.pass.cpp index 1a0b065f..40b40bfd 100644 --- a/test/std/containers/associative/multiset/multiset.cons/move_alloc.pass.cpp +++ b/test/std/containers/associative/multiset/multiset.cons/move_alloc.pass.cpp @@ -19,6 +19,7 @@ #include "../../../MoveOnly.h" #include "../../../test_compare.h" #include "test_allocator.h" +#include "Counter.h" int main() { @@ -137,5 +138,50 @@ int main() assert(m3.key_comp() == C(5)); assert(m1.empty()); } + { + typedef Counter V; + typedef std::less C; + typedef test_allocator A; + typedef std::multiset M; + typedef V* I; + Counter_base::gConstructed = 0; + { + V a1[] = + { + V(1), + V(1), + V(1), + V(2), + V(2), + V(2), + V(3), + V(3), + V(3) + }; + const size_t num = sizeof(a1)/sizeof(a1[0]); + assert(Counter_base::gConstructed == num); + + M m1(I(a1), I(a1+num), C(), A()); + assert(Counter_base::gConstructed == 2*num); + + M m2(m1); + assert(m2 == m1); + assert(Counter_base::gConstructed == 3*num); + + M m3(std::move(m1), A()); + assert(m3 == m2); + assert(m1.empty()); + assert(Counter_base::gConstructed == 3*num); + + { + M m4(std::move(m2), A(5)); + assert(Counter_base::gConstructed == 3*num); + assert(m4 == m3); + assert(m2.empty()); + } + assert(Counter_base::gConstructed == 2*num); + } + assert(Counter_base::gConstructed == 0); + } #endif // _LIBCPP_HAS_NO_RVALUE_REFERENCES } diff --git a/test/std/containers/associative/set/set.cons/move_alloc.pass.cpp b/test/std/containers/associative/set/set.cons/move_alloc.pass.cpp index c4617f0b..bad5e8f8 100644 --- a/test/std/containers/associative/set/set.cons/move_alloc.pass.cpp +++ b/test/std/containers/associative/set/set.cons/move_alloc.pass.cpp @@ -19,6 +19,7 @@ #include "../../../MoveOnly.h" #include "../../../test_compare.h" #include "test_allocator.h" +#include "Counter.h" int main() { @@ -137,5 +138,51 @@ int main() assert(m3.key_comp() == C(5)); assert(m1.empty()); } + { + typedef Counter V; + typedef std::less C; + typedef test_allocator A; + typedef std::set M; + typedef V* I; + Counter_base::gConstructed = 0; + { + V a1[] = + { + V(1), + V(1), + V(1), + V(2), + V(2), + V(2), + V(3), + V(3), + V(3) + }; + const size_t num = sizeof(a1)/sizeof(a1[0]); + assert(Counter_base::gConstructed == num); + + M m1(I(a1), I(a1+num), C(), A()); + assert(Counter_base::gConstructed == 3+num); + + M m2(m1); + assert(m2 == m1); + assert(Counter_base::gConstructed == 6+num); + + M m3(std::move(m1), A()); + assert(m3 == m2); + assert(m1.empty()); + assert(Counter_base::gConstructed == 6+num); + + { + M m4(std::move(m2), A(5)); + assert(Counter_base::gConstructed == 6+num); + assert(m4 == m3); + assert(m2.empty()); + } + assert(Counter_base::gConstructed == 3+num); + } + assert(Counter_base::gConstructed == 0); + } + #endif // _LIBCPP_HAS_NO_RVALUE_REFERENCES } diff --git a/test/support/Counter.h b/test/support/Counter.h new file mode 100644 index 00000000..2bc3642f --- /dev/null +++ b/test/support/Counter.h @@ -0,0 +1,52 @@ +//===----------------------------------------------------------------------===// +// +// The LLVM Compiler Infrastructure +// +// This file is dual licensed under the MIT and the University of Illinois Open +// Source Licenses. See LICENSE.TXT for details. +// +//===----------------------------------------------------------------------===// + +#ifndef COUNTER_H +#define COUNTER_H + +#include // for std::hash + +struct Counter_base { static int gConstructed; }; + +template +class Counter : public Counter_base +{ +public: + Counter() : data_() { ++gConstructed; } + Counter(const T &data) : data_(data) { ++gConstructed; } + Counter(const Counter& rhs) : data_(rhs.data_) { ++gConstructed; } + Counter& operator=(const Counter& rhs) { ++gConstructed; data_ = rhs.data_; return *this; } +#ifndef _LIBCPP_HAS_NO_RVALUE_REFERENCES + Counter(Counter&& rhs) : data_(std::move(rhs.data_)) { ++gConstructed; } + Counter& operator=(Counter&& rhs) { ++gConstructed; data_ = std::move(rhs.data_); return *this; } +#endif + ~Counter() { --gConstructed; } + + const T& get() const {return data_;} + + bool operator==(const Counter& x) const {return data_ == x.data_;} + bool operator< (const Counter& x) const {return data_ < x.data_;} + +private: + T data_; +}; + +int Counter_base::gConstructed = 0; + +namespace std { + +template +struct hash > + : public std::unary_function, std::size_t> +{ + std::size_t operator()(const Counter& x) const {return std::hash(x.get());} +}; +} + +#endif