public inbox for libstdc++-cvs@sourceware.org help / color / mirror / Atom feed
From: Tamar Christina <tnfchris@gcc.gnu.org> To: gcc-cvs@gcc.gnu.org, libstdc++-cvs@gcc.gnu.org Subject: [gcc(refs/vendors/ARM/heads/arm-perf-staging)] libstdc++: Add a move-only testsuite iterator type Date: Fri, 17 Jul 2020 13:36:07 +0000 (GMT) [thread overview] Message-ID: <20200717133607.D4B663AAB4B3@sourceware.org> (raw) https://gcc.gnu.org/g:d6d4b339f5b77070ee9ae896c2f20ba315b7b7e0 commit d6d4b339f5b77070ee9ae896c2f20ba315b7b7e0 Author: Patrick Palka <ppalka@redhat.com> Date: Mon Mar 2 16:21:33 2020 -0500 libstdc++: Add a move-only testsuite iterator type This adds a move-only testsuite iterator wrapper to <testsuite_iterators.h> which will be used in the tests for LWG 3355. The tests for LWG 3389 and 3390 are adjusted to use this new iterator wrapper. libstdc++-v3/ChangeLog: * testsuite/util/testsuite_iterators.h (input_iterator_wrapper_nocopy): New testsuite iterator. * testsuite/24_iterators/counted_iterator/lwg3389.cc: Use it. * testsuite/24_iterators/move_iterator/lwg3390.cc: Likewise. Diff: --- libstdc++-v3/ChangeLog | 5 ++++ .../24_iterators/counted_iterator/lwg3389.cc | 35 ++-------------------- .../24_iterators/move_iterator/lwg3390.cc | 35 ++-------------------- libstdc++-v3/testsuite/util/testsuite_iterators.h | 28 +++++++++++++++++ 4 files changed, 37 insertions(+), 66 deletions(-) diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog index 45ff06d4ea7..1d68f7ceaac 100644 --- a/libstdc++-v3/ChangeLog +++ b/libstdc++-v3/ChangeLog @@ -1,5 +1,10 @@ 2020-03-04 Patrick Palka <ppalka@redhat.com> + * testsuite/util/testsuite_iterators.h (input_iterator_wrapper_nocopy): + New testsuite iterator. + * testsuite/24_iterators/counted_iterator/lwg3389.cc: use it. + * testsuite/24_iterators/move_iterator/lwg3390.cc: Likewise. + * include/bits/ranges_uninitialized.h (uninitialized_copy_fn::operator()): Pass a reference type as the first argument to is_nothrow_assignable_v. diff --git a/libstdc++-v3/testsuite/24_iterators/counted_iterator/lwg3389.cc b/libstdc++-v3/testsuite/24_iterators/counted_iterator/lwg3389.cc index cf74fd47bec..8b9bf53f6c5 100644 --- a/libstdc++-v3/testsuite/24_iterators/counted_iterator/lwg3389.cc +++ b/libstdc++-v3/testsuite/24_iterators/counted_iterator/lwg3389.cc @@ -23,44 +23,13 @@ #include <testsuite_iterators.h> using __gnu_test::test_range; -using __gnu_test::input_iterator_wrapper; - -template<typename T> -struct move_only_wrapper : input_iterator_wrapper<T> -{ - using input_iterator_wrapper<T>::input_iterator_wrapper; - - move_only_wrapper() - : input_iterator_wrapper<T>(nullptr, nullptr) - { } - - move_only_wrapper(const move_only_wrapper&) = delete; - move_only_wrapper& - operator=(const move_only_wrapper&) = delete; - - move_only_wrapper(move_only_wrapper&&) = default; - move_only_wrapper& - operator=(move_only_wrapper&&) = default; - - using input_iterator_wrapper<T>::operator++; - - move_only_wrapper& - operator++() - { - input_iterator_wrapper<T>::operator++(); - return *this; - } -}; - -static_assert(std::input_iterator<move_only_wrapper<int>>); -static_assert(!std::forward_iterator<move_only_wrapper<int>>); -static_assert(!std::copyable<move_only_wrapper<int>>); +using __gnu_test::input_iterator_wrapper_nocopy; // LWG 3389 void test01() { int x[] = {1,2,3,4}; - test_range<int, move_only_wrapper> rx(x); + test_range<int, input_iterator_wrapper_nocopy> rx(x); auto it = std::counted_iterator(rx.begin(), 2); } diff --git a/libstdc++-v3/testsuite/24_iterators/move_iterator/lwg3390.cc b/libstdc++-v3/testsuite/24_iterators/move_iterator/lwg3390.cc index 1df7caccece..7e9f4a0d0cc 100644 --- a/libstdc++-v3/testsuite/24_iterators/move_iterator/lwg3390.cc +++ b/libstdc++-v3/testsuite/24_iterators/move_iterator/lwg3390.cc @@ -23,44 +23,13 @@ #include <testsuite_iterators.h> using __gnu_test::test_range; -using __gnu_test::input_iterator_wrapper; - -template<typename T> -struct move_only_wrapper : input_iterator_wrapper<T> -{ - using input_iterator_wrapper<T>::input_iterator_wrapper; - - move_only_wrapper() - : input_iterator_wrapper<T>(nullptr, nullptr) - { } - - move_only_wrapper(const move_only_wrapper&) = delete; - move_only_wrapper& - operator=(const move_only_wrapper&) = delete; - - move_only_wrapper(move_only_wrapper&&) = default; - move_only_wrapper& - operator=(move_only_wrapper&&) = default; - - using input_iterator_wrapper<T>::operator++; - - move_only_wrapper& - operator++() - { - input_iterator_wrapper<T>::operator++(); - return *this; - } -}; - -static_assert(std::input_iterator<move_only_wrapper<int>>); -static_assert(!std::forward_iterator<move_only_wrapper<int>>); -static_assert(!std::copyable<move_only_wrapper<int>>); +using __gnu_test::input_iterator_wrapper_nocopy; // LWG 3390 void test01() { int x[] = {1,2,3,4}; - test_range<int, move_only_wrapper> rx(x); + test_range<int, input_iterator_wrapper_nocopy> rx(x); auto it = std::make_move_iterator(rx.begin()); } diff --git a/libstdc++-v3/testsuite/util/testsuite_iterators.h b/libstdc++-v3/testsuite/util/testsuite_iterators.h index 417dff23c50..e47b2b03e40 100644 --- a/libstdc++-v3/testsuite/util/testsuite_iterators.h +++ b/libstdc++-v3/testsuite/util/testsuite_iterators.h @@ -674,6 +674,34 @@ namespace __gnu_test { return iter -= n; } }; + // A move-only input iterator type. + template<typename T> + struct input_iterator_wrapper_nocopy : input_iterator_wrapper<T> + { + using input_iterator_wrapper<T>::input_iterator_wrapper; + + input_iterator_wrapper_nocopy() + : input_iterator_wrapper<T>(nullptr, nullptr) + { } + + input_iterator_wrapper_nocopy(const input_iterator_wrapper_nocopy&) = delete; + input_iterator_wrapper_nocopy& + operator=(const input_iterator_wrapper_nocopy&) = delete; + + input_iterator_wrapper_nocopy(input_iterator_wrapper_nocopy&&) = default; + input_iterator_wrapper_nocopy& + operator=(input_iterator_wrapper_nocopy&&) = default; + + using input_iterator_wrapper<T>::operator++; + + input_iterator_wrapper_nocopy& + operator++() + { + input_iterator_wrapper<T>::operator++(); + return *this; + } + }; + // A type meeting the minimum std::range requirements template<typename T, template<typename> class Iter> class test_range
reply other threads:[~2020-07-17 13:36 UTC|newest] Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20200717133607.D4B663AAB4B3@sourceware.org \ --to=tnfchris@gcc.gnu.org \ --cc=gcc-cvs@gcc.gnu.org \ --cc=libstdc++-cvs@gcc.gnu.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).