diff --git a/libstdc++-v3/include/std/chrono b/libstdc++-v3/include/std/chrono index 9fc8f560d99..fb251848da8 100644 --- a/libstdc++-v3/include/std/chrono +++ b/libstdc++-v3/include/std/chrono @@ -94,13 +94,9 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION (_Period1::den / __gcd_den::value) * _Period2::den>; public: - using type = chrono::duration<__cr, __r>; + using type = chrono::duration<__cr, typename __r::type>; }; - template - struct __duration_common_type<__failure_type, _Period1, _Period2> - { typedef __failure_type type; }; - /// @endcond /// Specialization of common_type for chrono::duration types. @@ -108,9 +104,24 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION template struct common_type, chrono::duration<_Rep2, _Period2>> - : __duration_common_type, _Period1, _Period2> + : __duration_common_type, + typename _Period1::type, + typename _Period2::type> { }; + /// Specialization of common_type for two identical chrono::duration types. + /// @relates duration + template + struct common_type, + chrono::duration<_Rep, _Period>> + { using type = chrono::duration<_Rep, typename _Period::type>; }; + + /// Specialization of common_type for one chrono::duration type. + /// @relates duration + template + struct common_type> + { using type = chrono::duration<_Rep, typename _Period::type>; }; + // 20.11.4.3 specialization of common_type (for time_point, sfinae-friendly) /// @cond undocumented @@ -135,6 +146,19 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION : __timepoint_common_type, _Clock> { }; + /// Specialization of common_type for two identical chrono::time_point types. + /// @relates time_point + template + struct common_type, + chrono::time_point<_Clock, _Duration>> + { using type = chrono::time_point<_Clock, _Duration>; }; + + /// Specialization of common_type for one chrono::time_point type. + /// @relates time_point + template + struct common_type> + { using type = chrono::time_point<_Clock, _Duration>; }; + // @} group chrono namespace chrono @@ -401,8 +425,8 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION public: - typedef _Rep rep; - typedef _Period period; + using rep = _Rep; + using period = typename _Period::type; static_assert(!__is_duration<_Rep>::value, "rep cannot be a duration"); static_assert(__is_ratio<_Period>::value, @@ -438,11 +462,12 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION { return __r; } // 20.11.5.3 arithmetic - constexpr duration + + constexpr duration operator+() const { return *this; } - constexpr duration + constexpr duration operator-() const { return duration(-__r); } diff --git a/libstdc++-v3/testsuite/20_util/duration/requirements/reduced_period.cc b/libstdc++-v3/testsuite/20_util/duration/requirements/reduced_period.cc new file mode 100644 index 00000000000..9eb38a0e56f --- /dev/null +++ b/libstdc++-v3/testsuite/20_util/duration/requirements/reduced_period.cc @@ -0,0 +1,131 @@ +// Copyright (C) 2020 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-do compile { target c++11 } } + +// Test the changes introduced by P0548R1 "common_type and duration". +// Specifically, duration::period should be the reduced ratio, +// and common_type::type should be a duration using the +// reduced ratio. + +#include + +using std::chrono::duration; +using std::ratio; +using std::common_type; +using std::is_same; + +void +test01() +{ + using D1 = duration>; + static_assert( is_same>::value, + "duration::period is P::type, not P" ); + + using C1 = common_type::type; + static_assert( is_same>::value, + "common_type_t is duration"); + static_assert( is_same::type, C1>::value, + "common_type_t is common_type_t" ); + static_assert( is_same::type, C1>::value, + "common_type_t is common_type_t" ); + + using D2 = duration>; + static_assert( is_same>::value, + "duration::period is P2::type, not P2" ); + + using C2 = common_type::type; + static_assert( is_same>::value, + "common_type_t is duration"); + static_assert( is_same::type, C2>::value, + "common_type_t is common_type_t" ); + static_assert( is_same::type, C2>::value, + "common_type_t is common_type_t" ); + + using D3 = duration>; + static_assert( is_same>::value, + "duration::period is P3::type, not P3" ); + + using C3 = common_type::type; + static_assert( is_same>::value, + "common_type_t is duration"); + static_assert( is_same::type, C3>::value, + "common_type_t is common_type_t" ); + static_assert( is_same::type, C3>::value, + "common_type_t is common_type_t" ); + + using C12 = common_type::type; + static_assert( is_same::value, + "common_type_t uses the right period" ); + using C21 = common_type::type; + static_assert( is_same::value, + "common_type_t is common_type_t" ); + + using C13 = common_type::type; + static_assert( is_same::value, + "common_type_t uses the right period" ); + using C31 = common_type::type; + static_assert( is_same::value, + "common_type_t is common_type_t" ); + + using C23 = common_type::type; + static_assert( is_same::value, + "common_type_t uses the right period" ); + using C32 = common_type::type; + static_assert( is_same::value, + "common_type_t is common_type_t" ); + + using C123 = common_type::type; + static_assert( is_same::value, + "common_type of three durations uses the right period" ); + using C132 = common_type::type; + static_assert( is_same::value, "order doesn't matter" ); + using C312 = common_type::type; + static_assert( is_same::value, "order doesn't matter" ); + using C321 = common_type::type; + static_assert( is_same::value, "order doesn't matter" ); + + using C = common_type>, + duration>>::type; + static_assert( is_same::type, + ratio<1, 6>>>::value, "" ); +} + +void +test02() +{ + using D1 = duration>; + D1 d1; + static_assert( is_same::type>::value, + "unary + returns the reduced duration" ); + static_assert( is_same::type>::value, + "unary - returns the reduced duration" ); + + using D2 = duration>; + D2 d2; + static_assert( is_same::type>::value, + "unary + returns the reduced duration" ); + static_assert( is_same::type>::value, + "unary - returns the reduced duration" ); + + using D3 = duration>; + D3 d3; + static_assert( is_same::type>::value, + "unary + returns the reduced duration" ); + static_assert( is_same::type>::value, + "unary - returns the reduced duration" ); +} diff --git a/libstdc++-v3/testsuite/20_util/duration/requirements/typedefs_neg2.cc b/libstdc++-v3/testsuite/20_util/duration/requirements/typedefs_neg2.cc index d86e89b632b..02dab73b0eb 100644 --- a/libstdc++-v3/testsuite/20_util/duration/requirements/typedefs_neg2.cc +++ b/libstdc++-v3/testsuite/20_util/duration/requirements/typedefs_neg2.cc @@ -31,4 +31,5 @@ void test01() } // { dg-error "must be a specialization of ratio" "" { target *-*-* } 0 } -// { dg-prune-output "not a member" } +// { dg-prune-output "'num' is not a member of 'int'" } +// { dg-prune-output "'int' is not a class, struct, or union type" }