From 90fc44ecfa4741a055ec3d35722e2624c2231f98 Mon Sep 17 00:00:00 2001 From: Jonathan Wakely Date: Wed, 4 Jul 2018 09:07:23 +0100 Subject: [PATCH] Optimize std::rotl and std::rotr, add test for std::popcount * include/std/bit (__rotl, __rotr): Avoid branch. (_If_is_unsigned_integer): Use remove_cv_t. * testsuite/26_numerics/bit/bitops.count/popcount.cc: New. From-SVN: r262378 --- libstdc++-v3/ChangeLog | 6 + libstdc++-v3/include/std/bit | 10 +- .../26_numerics/bit/bitops.count/popcount.cc | 108 ++++++++++++++++++ 3 files changed, 117 insertions(+), 7 deletions(-) create mode 100644 libstdc++-v3/testsuite/26_numerics/bit/bitops.count/popcount.cc diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog index f6c39b6361c..e3211bc819f 100644 --- a/libstdc++-v3/ChangeLog +++ b/libstdc++-v3/ChangeLog @@ -1,3 +1,9 @@ +2018-07-04 Jonathan Wakely + + * include/std/bit (__rotl, __rotr): Avoid branch. + (_If_is_unsigned_integer): Use remove_cv_t. + * testsuite/26_numerics/bit/bitops.count/popcount.cc: New. + 2018-07-03 Jonathan Wakely P0556R3 Integral power-of-2 operations, P0553R2 Bit operations diff --git a/libstdc++-v3/include/std/bit b/libstdc++-v3/include/std/bit index 76aa0957b56..ace88954030 100644 --- a/libstdc++-v3/include/std/bit +++ b/libstdc++-v3/include/std/bit @@ -46,9 +46,7 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION { constexpr auto _Nd = numeric_limits<_Tp>::digits; const unsigned __sN = __s % _Nd; - if (__sN) - return (__x << __sN) | (__x >> (_Nd - __sN)); - return __x; + return (__x << __sN) | (__x >> ((-__sN) % _Nd)); } template @@ -57,9 +55,7 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION { constexpr auto _Nd = numeric_limits<_Tp>::digits; const unsigned __sN = __s % _Nd; - if (__sN) - return (__x >> __sN) | (__x << (_Nd - __sN)); - return __x; + return (__x >> __sN) | (__x << ((-__sN) % _Nd)); } template @@ -237,7 +233,7 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION template using _If_is_unsigned_integer - = typename _If_is_unsigned_integer_type<_Tp, _Up>::type; + = typename _If_is_unsigned_integer_type, _Up>::type; #if ! __STRICT_ANSI__ // [bitops.rot], rotating diff --git a/libstdc++-v3/testsuite/26_numerics/bit/bitops.count/popcount.cc b/libstdc++-v3/testsuite/26_numerics/bit/bitops.count/popcount.cc new file mode 100644 index 00000000000..2982cb19bbe --- /dev/null +++ b/libstdc++-v3/testsuite/26_numerics/bit/bitops.count/popcount.cc @@ -0,0 +1,108 @@ +// Copyright (C) 2018 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 run { target c++11 } } + +// { dg-options "-std=gnu++2a" } +// { dg-do compile { target c++2a } } + +#include + +template +constexpr auto +test(UInt x) +-> decltype(std::popcount(x)) +{ + static_assert( noexcept(std::popcount(x)) ); + + constexpr unsigned digits = std::numeric_limits::digits; + + static_assert( std::popcount((UInt)0) == 0 ); + static_assert( std::popcount((UInt)-1) == digits ); + static_assert( std::popcount((UInt)-2) == digits - 1 ); + static_assert( std::popcount((UInt)127) == 7 ); + + static_assert( std::popcount((UInt)1) == 1 ); + static_assert( std::popcount((UInt)2) == 1 ); + static_assert( std::popcount((UInt)0x70) == 3 ); + + if constexpr (std::numeric_limits::digits > 8) + { + static_assert( std::popcount((UInt)(0x101)) == 2 ); + static_assert( std::popcount((UInt)(0xfff)) == 12 ); + } + + if constexpr (std::numeric_limits::digits > 64) + { + static_assert( std::popcount((UInt)0xffffffffffffffff) == 64 ); + static_assert( std::popcount(0x5555555555555555 | ((UInt)1 << 64)) == 33 ); + static_assert( std::popcount(0x5555555555555555 | ((UInt)3 << 64)) == 34 ); + } + + return true; +} + +static_assert( test( (unsigned char)0 ) ); +static_assert( test( (unsigned short)0 ) ); +static_assert( test( (unsigned int)0 ) ); +static_assert( test( (unsigned long)0 ) ); +static_assert( test( (unsigned long long)0 ) ); + +// std::popcount(T) shall not participate in overload resolution +// unless T is an unsigned integer type. +struct X { constexpr bool did_not_match() { return true; } }; +constexpr X test(...) { return X{}; } +static_assert( test( (bool)0 ).did_not_match() ); +static_assert( test( (char)0 ).did_not_match() ); +static_assert( test( (int)0 ).did_not_match() ); +static_assert( test( (char16_t)0 ).did_not_match() ); +static_assert( test( (float)0 ).did_not_match() ); +static_assert( test( (void*)0 ).did_not_match() ); +static_assert( test( X{} ).did_not_match() ); +enum E : unsigned { e }; +static_assert( test( e ).did_not_match() ); + +#ifndef __STRICT_ANSI__ +#include +static_assert( std::popcount(std::byte{0x00}) == 0 ); +static_assert( std::popcount(std::byte{0x01}) == 1 ); +static_assert( std::popcount(std::byte{0x02}) == 1 ); +static_assert( std::popcount(std::byte{0x03}) == 2 ); +static_assert( std::popcount(std::byte{0x30}) == 2 ); +static_assert( std::popcount(std::byte{0x40}) == 1 ); +static_assert( std::popcount(std::byte{0x41}) == 2 ); +static_assert( std::popcount(std::byte{0xff}) == 8 ); +#else +static_assert( test( (std::byte)0 ).did_not_match() ); +#endif + +#if !defined(__STRICT_ANSI__) && defined _GLIBCXX_USE_INT128 +static_assert( test( (unsigned __int128)0 ) ); +static_assert( test( (__int128)0 ).did_not_match() ); +#endif +#if defined(__GLIBCXX_TYPE_INT_N_0) +static_assert( test( (unsigned __GLIBCXX_TYPE_INT_N_0)0 ) ); +static_assert( test( (__GLIBCXX_TYPE_INT_N_0)0 ).did_not_match() ); +#endif +#if defined(__GLIBCXX_TYPE_INT_N_1) +static_assert( test( (unsigned __GLIBCXX_TYPE_INT_N_1)0 ) ); +static_assert( test( (__GLIBCXX_TYPE_INT_N_1)0 ).did_not_match() ); +#endif +#if defined(__GLIBCXX_TYPE_INT_N_2) +static_assert( test( (unsigned __GLIBCXX_TYPE_INT_N_2)0 ) ); +static_assert( test( (__GLIBCXX_TYPE_INT_N_2)0 ).did_not_match() ); +#endif -- 2.30.2