Skip to content

[libc] Make BigInt bit_cast-able to compatible types #74837

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Dec 8, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions libc/src/__support/UInt.h
Original file line number Diff line number Diff line change
Expand Up @@ -25,12 +25,13 @@

namespace LIBC_NAMESPACE::cpp {

// BigInt has the same semantics as the 'standard integer types' and can be
// safely 'bit_cast'ed to compatible types.
template <size_t Bits, bool Signed> struct BigInt {

static_assert(Bits > 0 && Bits % 64 == 0,
"Number of bits in BigInt should be a multiple of 64.");
LIBC_INLINE_VAR static constexpr size_t WORDCOUNT = Bits / 64;
uint64_t val[WORDCOUNT]{};
uint64_t val[WORDCOUNT];

LIBC_INLINE_VAR static constexpr uint64_t MASK32 = 0xFFFFFFFFu;

Expand Down
5 changes: 4 additions & 1 deletion libc/test/src/__support/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -89,8 +89,11 @@ add_libc_test(
SRCS
uint_test.cpp
DEPENDS
libc.src.__support.uint
libc.src.__support.CPP.bit
libc.src.__support.CPP.optional
libc.src.__support.CPP.type_traits
libc.src.__support.macros.properties.float
libc.src.__support.uint
)

add_libc_test(
Expand Down
70 changes: 61 additions & 9 deletions libc/test/src/__support/uint_test.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,23 +6,73 @@
//
//===----------------------------------------------------------------------===//

#include "src/__support/CPP/bit.h" // bit_cast
#include "src/__support/CPP/optional.h"
#include "src/__support/CPP/type_traits.h" // is_trivially_constructible
#include "src/__support/UInt.h"
#include "src/__support/macros/properties/float.h" // LIBC_COMPILER_HAS_FLOAT128

#include "test/UnitTest/Test.h"

// We want to test LIBC_NAMESPACE::cpp::UInt<128> explicitly. So, for
#include <math.h> // HUGE_VALF, HUGE_VALF

namespace LIBC_NAMESPACE {

using LL_UInt64 = cpp::UInt<64>;
// We want to test cpp::UInt<128> explicitly. So, for
// convenience, we use a sugar which does not conflict with the UInt128 type
// which can resolve to __uint128_t if the platform has it.
using LL_UInt128 = LIBC_NAMESPACE::cpp::UInt<128>;
using LL_UInt192 = LIBC_NAMESPACE::cpp::UInt<192>;
using LL_UInt256 = LIBC_NAMESPACE::cpp::UInt<256>;
using LL_UInt320 = LIBC_NAMESPACE::cpp::UInt<320>;
using LL_UInt512 = LIBC_NAMESPACE::cpp::UInt<512>;
using LL_UInt1024 = LIBC_NAMESPACE::cpp::UInt<1024>;
using LL_UInt128 = cpp::UInt<128>;
using LL_UInt192 = cpp::UInt<192>;
using LL_UInt256 = cpp::UInt<256>;
using LL_UInt320 = cpp::UInt<320>;
using LL_UInt512 = cpp::UInt<512>;
using LL_UInt1024 = cpp::UInt<1024>;

using LL_Int128 = cpp::Int<128>;
using LL_Int192 = cpp::Int<192>;

TEST(LlvmLibcUIntClassTest, BitCastToFromDouble) {
static_assert(cpp::is_trivially_constructible<LL_UInt64>::value);
static_assert(cpp::is_trivially_copyable<LL_UInt64>::value);
static_assert(sizeof(LL_UInt64) == sizeof(double));
const double inf = HUGE_VAL;
const double max = DBL_MAX;
const double array[] = {0.0, 0.1, 1.0, max, inf};
for (double value : array) {
LL_UInt64 back = cpp::bit_cast<LL_UInt64>(value);
double forth = cpp::bit_cast<double>(back);
EXPECT_TRUE(value == forth);
}
}

using LL_Int128 = LIBC_NAMESPACE::cpp::Int<128>;
using LL_Int192 = LIBC_NAMESPACE::cpp::Int<192>;
#ifdef __SIZEOF_INT128__
TEST(LlvmLibcUIntClassTest, BitCastToFromNativeUint128) {
static_assert(cpp::is_trivially_constructible<LL_UInt128>::value);
static_assert(cpp::is_trivially_copyable<LL_UInt128>::value);
static_assert(sizeof(LL_UInt128) == sizeof(__uint128_t));
const __uint128_t array[] = {0, 1, ~__uint128_t(0)};
for (__uint128_t value : array) {
LL_UInt128 back = cpp::bit_cast<LL_UInt128>(value);
__uint128_t forth = cpp::bit_cast<__uint128_t>(back);
EXPECT_TRUE(value == forth);
}
}
#endif

#ifdef LIBC_COMPILER_HAS_FLOAT128
TEST(LlvmLibcUIntClassTest, BitCastToFromNativeFloat128) {
static_assert(cpp::is_trivially_constructible<LL_UInt128>::value);
static_assert(cpp::is_trivially_copyable<LL_UInt128>::value);
static_assert(sizeof(LL_UInt128) == sizeof(float128));
const float128 array[] = {0, 0.1, 1};
for (float128 value : array) {
LL_UInt128 back = cpp::bit_cast<LL_UInt128>(value);
float128 forth = cpp::bit_cast<float128>(back);
EXPECT_TRUE(value == forth);
}
}
#endif

TEST(LlvmLibcUIntClassTest, BasicInit) {
LL_UInt128 half_val(12345);
Expand Down Expand Up @@ -634,3 +684,5 @@ TEST(LlvmLibcUIntClassTest, ConstructorFromUInt128Tests) {
}

#endif // __SIZEOF_INT128__

} // namespace LIBC_NAMESPACE
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,10 @@ libc_test(
name = "uint_test",
srcs = ["uint_test.cpp"],
deps = [
"//libc:__support_cpp_bit",
"//libc:__support_cpp_optional",
"//libc:__support_cpp_type_traits",
"//libc:__support_macros_properties_float",
"//libc:__support_uint",
],
)
Expand Down