From: Marshall Clow Date: Mon, 27 Nov 2017 22:27:22 +0000 (+0000) Subject: Fix problems with r'890 when building on machines where sizeof(size_t) != sizeof... X-Git-Tag: llvmorg-6.0.0-rc1~2576 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=ea44ee202f6a73bfd832d89bd882b34cda9ca65e;p=platform%2Fupstream%2Fllvm.git Fix problems with r'890 when building on machines where sizeof(size_t) != sizeof(unsigned long long) and C++03 llvm-svn: 319106 --- diff --git a/libcxx/include/bitset b/libcxx/include/bitset index e26e674..cd6c289 100644 --- a/libcxx/include/bitset +++ b/libcxx/include/bitset @@ -235,8 +235,13 @@ void __bitset<_N_words, _Size>::__init(unsigned long long __v, false_type) _NOEXCEPT { __storage_type __t[sizeof(unsigned long long) / sizeof(__storage_type)]; - for (size_t __i = 0; __i < sizeof(__t)/sizeof(__t[0]); ++__i, __v >>= __bits_per_word) - __t[__i] = static_cast<__storage_type>(__v); + size_t __sz = _Size; + for (size_t __i = 0; __i < sizeof(__t)/sizeof(__t[0]); ++__i, __v >>= __bits_per_word, __sz -= __bits_per_word ) + if ( __sz < __bits_per_word) + __t[__i] = static_cast<__storage_type>(__v) & ( 1ULL << __sz ) - 1; + else + __t[__i] = static_cast<__storage_type>(__v); + _VSTD::copy(__t, __t + sizeof(__t)/sizeof(__t[0]), __first_); _VSTD::fill(__first_ + sizeof(__t)/sizeof(__t[0]), __first_ + sizeof(__first_)/sizeof(__first_[0]), __storage_type(0)); @@ -248,6 +253,9 @@ void __bitset<_N_words, _Size>::__init(unsigned long long __v, true_type) _NOEXCEPT { __first_[0] = __v; + if (_Size < __bits_per_word) + __first_[0] &= ( 1ULL << _Size ) - 1; + _VSTD::fill(__first_ + 1, __first_ + sizeof(__first_)/sizeof(__first_[0]), __storage_type(0)); } @@ -261,7 +269,9 @@ __bitset<_N_words, _Size>::__bitset(unsigned long long __v) _NOEXCEPT #if __SIZEOF_SIZE_T__ == 8 : __first_{__v} #elif __SIZEOF_SIZE_T__ == 4 - : __first_{static_cast<__storage_type>(__v), static_cast<__storage_type>(__v >> __bits_per_word)} + : __first_{static_cast<__storage_type>(__v), + _Size >= 2 * __bits_per_word ? static_cast<__storage_type>(__v >> __bits_per_word) + : static_cast<__storage_type>((__v >> __bits_per_word) & (__storage_type(1) << (_Size - __bits_per_word)) - 1)} #else #error This constructor has not been ported to this platform #endif diff --git a/libcxx/test/std/utilities/template.bitset/bitset.members/to_ullong.pass.cpp b/libcxx/test/std/utilities/template.bitset/bitset.members/to_ullong.pass.cpp index fb35029..c4a818f 100644 --- a/libcxx/test/std/utilities/template.bitset/bitset.members/to_ullong.pass.cpp +++ b/libcxx/test/std/utilities/template.bitset/bitset.members/to_ullong.pass.cpp @@ -8,6 +8,7 @@ //===----------------------------------------------------------------------===// // test unsigned long long to_ullong() const; +extern "C" int printf(const char *, ...); #include #include @@ -37,7 +38,7 @@ void test_to_ullong() assert(j == v.to_ullong()); } { // test values bigger than can fit into the bitset - const unsigned long long val = 0xAAAAAAAAAAAAAAAAULL; + const unsigned long long val = 0x55AAAAFFFFAAAA55ULL; const bool canFit = N < sizeof(unsigned long long) * CHAR_BIT; const unsigned long long mask = canFit ? (1ULL << N) - 1 : (unsigned long long)(-1); std::bitset v(val); diff --git a/libcxx/test/std/utilities/template.bitset/bitset.members/to_ulong.pass.cpp b/libcxx/test/std/utilities/template.bitset/bitset.members/to_ulong.pass.cpp index 86b4815..536c272 100644 --- a/libcxx/test/std/utilities/template.bitset/bitset.members/to_ulong.pass.cpp +++ b/libcxx/test/std/utilities/template.bitset/bitset.members/to_ulong.pass.cpp @@ -16,12 +16,9 @@ #include #include -#include - template void test_to_ulong() { - std::cout << "Testing size = " << N << std::endl; const std::size_t M = sizeof(unsigned long) * CHAR_BIT < N ? sizeof(unsigned long) * CHAR_BIT : N; const bool is_M_zero = std::integral_constant::value; // avoid compiler warnings const std::size_t X = is_M_zero ? sizeof(unsigned long) * CHAR_BIT - 1 : sizeof(unsigned long) * CHAR_BIT - M; @@ -37,13 +34,12 @@ void test_to_ulong() for (std::size_t i = 0; i < sizeof(tests)/sizeof(tests[0]); ++i) { std::size_t j = tests[i]; - std::cout << " Testing value = " << j << std::endl; std::bitset v(j); assert(j == v.to_ulong()); } { // test values bigger than can fit into the bitset - const unsigned long val = 0xAAAAAAAAULL; + const unsigned long val = 0x5AFFFFA5ULL; const bool canFit = N < sizeof(unsigned long) * CHAR_BIT; const unsigned long mask = canFit ? (1ULL << N) - 1 : (unsigned long)(-1); std::bitset v(val);