Magic-Mirror/llvm-capstone
1
0
Fork 0
mirror of https://github.com/capstone-engine/llvm-capstone.git synced 2024-11-23 13:50:11 +00:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

Fix numeric_limits<XXX>::is_modulo for signed arithmetic types. We were reporting true, for all arithmetic types, which is incorrect. Fix the tests which were wrong, too. This fixes PR#20158.

Browse Source 
llvm-svn: 214371
This commit is contained in:
Marshall Clow 2014-07-31 01:18:05 +00:00
parent b9a23cdcdb
commit 6a577a819f
2 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
libcxx/include/limits
View File

@ -235,7 +235,7 @@ protected:
static _LIBCPP_CONSTEXPR const bool is_iec559 = false;
static _LIBCPP_CONSTEXPR const bool is_bounded = true;
static _LIBCPP_CONSTEXPR const bool is_modulo = true;
static _LIBCPP_CONSTEXPR const bool is_modulo = !_VSTD::is_signed<_Tp>::value;
#if __i386__ || __x86_64__
static _LIBCPP_CONSTEXPR const bool traps = true;

16
libcxx/test/language.support/support.limits/limits/numeric.limits.members/is_modulo.pass.cpp
View File

@ -26,24 +26,24 @@ test()
int main()
{
test<bool, false>();
test<char, true>();
test<signed char, true>();
// test<char, false>(); // don't know
test<signed char, false>();
test<unsigned char, true>();
test<wchar_t, true>();
// test<wchar_t, false>(); // don't know
#ifndef _LIBCPP_HAS_NO_UNICODE_CHARS
test<char16_t, true>();
test<char32_t, true>();
#endif // _LIBCPP_HAS_NO_UNICODE_CHARS
test<short, true>();
test<short, false>();
test<unsigned short, true>();
test<int, true>();
test<int, false>();
test<unsigned int, true>();
test<long, true>();
test<long, false>();
test<unsigned long, true>();
test<long long, true>();
test<long long, false>();
test<unsigned long long, true>();
#ifndef _LIBCPP_HAS_NO_INT128
test<__int128_t, true>();
test<__int128_t, false>();
test<__uint128_t, true>();
#endif
test<float, false>();