From: Jonathan Wakely Date: Tue, 4 Nov 2014 02:49:33 +0000 (+0000) Subject: Make reference_wrapper trivially copyable. X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=565115eaaf58eeb90dbcb2e6fe25db93342d3048;p=gcc.git Make reference_wrapper trivially copyable. * include/std/functional (reference_wrapper): Define copy constructor and copy assignment as defaulted. * testsuite/20_util/bind/ref_neg.cc: Adjust dg-error. * testsuite/20_util/reference_wrapper/requirements.cc: New. From-SVN: r217067 --- diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog index 19a86a08cf5..4f2c9b5ab65 100644 --- a/libstdc++-v3/ChangeLog +++ b/libstdc++-v3/ChangeLog @@ -11,6 +11,11 @@ * include/bits/regex.tcc (regex_traits::lookup_classname): Use ctype_base::blank. + * include/std/functional (reference_wrapper): Define copy constructor + and copy assignment as defaulted. + * testsuite/20_util/bind/ref_neg.cc: Adjust dg-error. + * testsuite/20_util/reference_wrapper/requirements.cc: New. + 2014-11-03 Paolo Carlini * include/parallel/algo.h: Do not use default arguments in function diff --git a/libstdc++-v3/include/std/functional b/libstdc++-v3/include/std/functional index f8e9b54fd91..25ec1b351b8 100644 --- a/libstdc++-v3/include/std/functional +++ b/libstdc++-v3/include/std/functional @@ -410,16 +410,10 @@ _GLIBCXX_HAS_NESTED_TYPE(result_type) reference_wrapper(_Tp&&) = delete; - reference_wrapper(const reference_wrapper<_Tp>& __inref) noexcept - : _M_data(__inref._M_data) - { } + reference_wrapper(const reference_wrapper&) = default; reference_wrapper& - operator=(const reference_wrapper<_Tp>& __inref) noexcept - { - _M_data = __inref._M_data; - return *this; - } + operator=(const reference_wrapper&) = default; operator _Tp&() const noexcept { return this->get(); } diff --git a/libstdc++-v3/testsuite/20_util/bind/ref_neg.cc b/libstdc++-v3/testsuite/20_util/bind/ref_neg.cc index 1063dd35d7e..4e627cfaf48 100644 --- a/libstdc++-v3/testsuite/20_util/bind/ref_neg.cc +++ b/libstdc++-v3/testsuite/20_util/bind/ref_neg.cc @@ -30,10 +30,10 @@ void test01() { const int dummy = 0; std::bind(&inc, _1)(0); // { dg-error "no match" } - // { dg-error "rvalue|const" "" { target *-*-* } 1213 } - // { dg-error "rvalue|const" "" { target *-*-* } 1227 } - // { dg-error "rvalue|const" "" { target *-*-* } 1241 } - // { dg-error "rvalue|const" "" { target *-*-* } 1255 } + // { dg-error "rvalue|const" "" { target *-*-* } 1207 } + // { dg-error "rvalue|const" "" { target *-*-* } 1221 } + // { dg-error "rvalue|const" "" { target *-*-* } 1235 } + // { dg-error "rvalue|const" "" { target *-*-* } 1249 } std::bind(&inc, std::ref(dummy))(); // { dg-error "no match" } } diff --git a/libstdc++-v3/testsuite/20_util/reference_wrapper/requirements.cc b/libstdc++-v3/testsuite/20_util/reference_wrapper/requirements.cc new file mode 100644 index 00000000000..c4d784cf59d --- /dev/null +++ b/libstdc++-v3/testsuite/20_util/reference_wrapper/requirements.cc @@ -0,0 +1,32 @@ +// Copyright (C) 2014 Free Software Foundation, Inc. +// +// This file is part of the GNU ISO C++ Library. This library is free +// software; you can redistribute it and/or modify it under the +// terms of the GNU General Public License as published by the +// Free Software Foundation; either version 3, or (at your option) +// any later version. + +// This library is distributed in the hope that it will be useful, +// but WITHOUT ANY WARRANTY; without even the implied warranty of +// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +// GNU General Public License for more details. + +// You should have received a copy of the GNU General Public License along +// with this library; see the file COPYING3. If not see +// . + +// { dg-options "-std=gnu++11" } +// { dg-do compile } + +#include + +struct NonTrivial +{ + ~NonTrivial() { } +}; + +using R = std::reference_wrapper; + +static_assert(std::is_copy_constructible::value, "copy constructible"); +static_assert(std::is_copy_assignable::value, "copy assignable"); +static_assert(std::is_trivially_copyable::value, "trivially copyable");