[libcxx] r332901 - Implement deduction guides for vector
Marshall Clow via cfe-commits
cfe-commits at lists.llvm.org
Mon May 21 14:30:12 PDT 2018
Author: marshall
Date: Mon May 21 14:30:12 2018
New Revision: 332901
URL: http://llvm.org/viewvc/llvm-project?rev=332901&view=rev
Log:
Implement deduction guides for vector
Added:
libcxx/trunk/test/std/containers/sequences/vector/vector.cons/deduct.fail.cpp
libcxx/trunk/test/std/containers/sequences/vector/vector.cons/deduct.pass.cpp
Modified:
libcxx/trunk/include/vector
Modified: libcxx/trunk/include/vector
URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/include/vector?rev=332901&r1=332900&r2=332901&view=diff
==============================================================================
--- libcxx/trunk/include/vector (original)
+++ libcxx/trunk/include/vector Mon May 21 14:30:12 2018
@@ -244,6 +244,10 @@ public:
bool __invariants() const;
};
+template <class InputIterator, class Allocator = allocator<typename iterator_traits<InputIterator>::value_type>>
+ vector(InputIterator, InputIterator, Allocator = Allocator())
+ -> vector<typename iterator_traits<InputIterator>::value_type, Allocator>;
+
template <class Allocator> struct hash<std::vector<bool, Allocator>>;
template <class T, class Allocator> bool operator==(const vector<T,Allocator>& x, const vector<T,Allocator>& y);
@@ -316,13 +320,14 @@ template <class _Tp, class _Allocator>
class __vector_base
: protected __vector_base_common<true>
{
-protected:
- typedef _Tp value_type;
+public:
typedef _Allocator allocator_type;
typedef allocator_traits<allocator_type> __alloc_traits;
+ typedef typename __alloc_traits::size_type size_type;
+protected:
+ typedef _Tp value_type;
typedef value_type& reference;
typedef const value_type& const_reference;
- typedef typename __alloc_traits::size_type size_type;
typedef typename __alloc_traits::difference_type difference_type;
typedef typename __alloc_traits::pointer pointer;
typedef typename __alloc_traits::const_pointer const_pointer;
@@ -492,8 +497,8 @@ public:
#if _LIBCPP_STD_VER > 11
explicit vector(size_type __n, const allocator_type& __a);
#endif
- vector(size_type __n, const_reference __x);
- vector(size_type __n, const_reference __x, const allocator_type& __a);
+ vector(size_type __n, const value_type& __x);
+ vector(size_type __n, const value_type& __x, const allocator_type& __a);
template <class _InputIterator>
vector(_InputIterator __first,
typename enable_if<__is_input_iterator <_InputIterator>::value &&
@@ -890,6 +895,22 @@ private:
};
+#ifndef _LIBCPP_HAS_NO_DEDUCTION_GUIDES
+template<class _InputIterator,
+ class _Alloc = typename std::allocator<typename iterator_traits<_InputIterator>::value_type>,
+ class = typename enable_if<__is_allocator<_Alloc>::value, void>::type
+ >
+vector(_InputIterator, _InputIterator)
+ -> vector<typename iterator_traits<_InputIterator>::value_type, _Alloc>;
+
+template<class _InputIterator,
+ class _Alloc,
+ class = typename enable_if<__is_allocator<_Alloc>::value, void>::type
+ >
+vector(_InputIterator, _InputIterator, _Alloc)
+ -> vector<typename iterator_traits<_InputIterator>::value_type, _Alloc>;
+#endif
+
template <class _Tp, class _Allocator>
void
vector<_Tp, _Allocator>::__swap_out_circular_buffer(__split_buffer<value_type, allocator_type&>& __v)
@@ -1099,7 +1120,7 @@ vector<_Tp, _Allocator>::vector(size_typ
#endif
template <class _Tp, class _Allocator>
-vector<_Tp, _Allocator>::vector(size_type __n, const_reference __x)
+vector<_Tp, _Allocator>::vector(size_type __n, const value_type& __x)
{
#if _LIBCPP_DEBUG_LEVEL >= 2
__get_db()->__insert_c(this);
@@ -1112,7 +1133,7 @@ vector<_Tp, _Allocator>::vector(size_typ
}
template <class _Tp, class _Allocator>
-vector<_Tp, _Allocator>::vector(size_type __n, const_reference __x, const allocator_type& __a)
+vector<_Tp, _Allocator>::vector(size_type __n, const value_type& __x, const allocator_type& __a)
: __base(__a)
{
#if _LIBCPP_DEBUG_LEVEL >= 2
Added: libcxx/trunk/test/std/containers/sequences/vector/vector.cons/deduct.fail.cpp
URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/containers/sequences/vector/vector.cons/deduct.fail.cpp?rev=332901&view=auto
==============================================================================
--- libcxx/trunk/test/std/containers/sequences/vector/vector.cons/deduct.fail.cpp (added)
+++ libcxx/trunk/test/std/containers/sequences/vector/vector.cons/deduct.fail.cpp Mon May 21 14:30:12 2018
@@ -0,0 +1,40 @@
+//===----------------------------------------------------------------------===//
+//
+// 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.
+//
+//===----------------------------------------------------------------------===//
+
+// <vector>
+// UNSUPPORTED: c++98, c++03, c++11, c++14
+// UNSUPPORTED: libcpp-no-deduction-guides
+
+
+// template <class InputIterator, class Allocator = allocator<typename iterator_traits<InputIterator>::value_type>>
+// vector(InputIterator, InputIterator, Allocator = Allocator())
+// -> vector<typename iterator_traits<InputIterator>::value_type, Allocator>;
+//
+
+
+#include <deque>
+#include <iterator>
+#include <cassert>
+#include <cstddef>
+
+
+int main()
+{
+// Test the explicit deduction guides
+
+// Test the implicit deduction guides
+ {
+// vector (allocator &)
+ std::vector vec((std::allocator<int>())); // expected-error {{no viable constructor or deduction guide for deduction of template arguments of 'vector'}}
+// Note: The extra parens are necessary, since otherwise clang decides it is a function declaration.
+// Also, we can't use {} instead of parens, because that constructs a
+// deque<allocator<int>, allocator<allocator<int>>>
+ }
+
+}
Added: libcxx/trunk/test/std/containers/sequences/vector/vector.cons/deduct.pass.cpp
URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/containers/sequences/vector/vector.cons/deduct.pass.cpp?rev=332901&view=auto
==============================================================================
--- libcxx/trunk/test/std/containers/sequences/vector/vector.cons/deduct.pass.cpp (added)
+++ libcxx/trunk/test/std/containers/sequences/vector/vector.cons/deduct.pass.cpp Mon May 21 14:30:12 2018
@@ -0,0 +1,116 @@
+//===----------------------------------------------------------------------===//
+//
+// 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.
+//
+//===----------------------------------------------------------------------===//
+
+// <vector>
+// UNSUPPORTED: c++98, c++03, c++11, c++14
+// UNSUPPORTED: libcpp-no-deduction-guides
+
+
+// template <class InputIterator, class Allocator = allocator<typename iterator_traits<InputIterator>::value_type>>
+// deque(InputIterator, InputIterator, Allocator = Allocator())
+// -> deque<typename iterator_traits<InputIterator>::value_type, Allocator>;
+//
+
+
+#include <vector>
+#include <iterator>
+#include <cassert>
+#include <cstddef>
+#include <climits> // INT_MAX
+
+#include "test_macros.h"
+#include "test_iterators.h"
+#include "test_allocator.h"
+
+struct A {};
+
+int main()
+{
+
+// Test the explicit deduction guides
+ {
+ const int arr[] = {0, 1, 2, 3, 4, 5, 6, 7, 8, 9 };
+ std::vector vec(std::begin(arr), std::end(arr));
+
+ static_assert(std::is_same_v<decltype(vec), std::vector<int>>, "");
+ assert(std::equal(vec.begin(), vec.end(), std::begin(arr), std::end(arr)));
+ }
+
+ {
+ const long arr[] = {INT_MAX, 1L, 2L, 3L };
+ std::vector vec(std::begin(arr), std::end(arr), std::allocator<long>());
+ static_assert(std::is_same_v<decltype(vec)::value_type, long>, "");
+ assert(vec.size() == 4);
+ assert(vec[0] == INT_MAX);
+ assert(vec[1] == 1L);
+ assert(vec[2] == 2L);
+ }
+
+// Test the implicit deduction guides
+
+ {
+// We don't expect this one to work.
+// std::vector vec(std::allocator<int>()); // vector (allocator &)
+ }
+
+ {
+ std::vector vec(1, A{}); // vector (size_type, T)
+ static_assert(std::is_same_v<decltype(vec)::value_type, A>, "");
+ static_assert(std::is_same_v<decltype(vec)::allocator_type, std::allocator<A>>, "");
+ assert(vec.size() == 1);
+ }
+
+ {
+ std::vector vec(1, A{}, test_allocator<A>()); // vector (size_type, T, allocator)
+ static_assert(std::is_same_v<decltype(vec)::value_type, A>, "");
+ static_assert(std::is_same_v<decltype(vec)::allocator_type, test_allocator<A>>, "");
+ assert(vec.size() == 1);
+ }
+
+ {
+ std::vector vec{1U, 2U, 3U, 4U, 5U}; // vector(initializer-list)
+ static_assert(std::is_same_v<decltype(vec)::value_type, unsigned>, "");
+ assert(vec.size() == 5);
+ assert(vec[2] == 3U);
+ }
+
+ {
+ std::vector vec({1.0, 2.0, 3.0, 4.0}, test_allocator<double>()); // vector(initializer-list, allocator)
+ static_assert(std::is_same_v<decltype(vec)::value_type, double>, "");
+ static_assert(std::is_same_v<decltype(vec)::allocator_type, test_allocator<double>>, "");
+ assert(vec.size() == 4);
+ assert(vec[3] == 4.0);
+ }
+
+ {
+ std::vector<long double> source;
+ std::vector vec(source); // vector(vector &)
+ static_assert(std::is_same_v<decltype(vec)::value_type, long double>, "");
+ static_assert(std::is_same_v<decltype(vec)::allocator_type, std::allocator<long double>>, "");
+ assert(vec.size() == 0);
+ }
+
+
+// A couple of vector<bool> tests, too!
+ {
+ std::vector vec(3, true); // vector(initializer-list)
+ static_assert(std::is_same_v<decltype(vec)::value_type, bool>, "");
+ static_assert(std::is_same_v<decltype(vec)::allocator_type, std::allocator<bool>>, "");
+ assert(vec.size() == 3);
+ assert(vec[0] && vec[1] && vec[2]);
+ }
+
+ {
+ std::vector<bool> source;
+ std::vector vec(source); // vector(vector &)
+ static_assert(std::is_same_v<decltype(vec)::value_type, bool>, "");
+ static_assert(std::is_same_v<decltype(vec)::allocator_type, std::allocator<bool>>, "");
+ assert(vec.size() == 0);
+ }
+}
More information about the cfe-commits
mailing list