From 89c15dd115d136bbfc20aeddef87ed2d87972e77 Mon Sep 17 00:00:00 2001 From: ReinUsesLisp Date: Fri, 15 Jan 2021 04:05:22 -0300 Subject: common/alignment: Upgrade to use constraints instead of static asserts --- src/common/alignment.h | 22 +++++++++------------- 1 file changed, 9 insertions(+), 13 deletions(-) diff --git a/src/common/alignment.h b/src/common/alignment.h index 2127dc186..fb81f10d8 100644 --- a/src/common/alignment.h +++ b/src/common/alignment.h @@ -9,8 +9,7 @@ namespace Common { template -[[nodiscard]] constexpr T AlignUp(T value, std::size_t size) { - static_assert(std::is_unsigned_v, "T must be an unsigned value."); +requires std::is_unsigned_v[[nodiscard]] constexpr T AlignUp(T value, size_t size) { auto mod{static_cast(value % size)}; value -= mod; return static_cast(mod == T{0} ? value : value + size); @@ -22,36 +21,33 @@ requires std::is_unsigned_v[[nodiscard]] constexpr T AlignUpLog2(T value, siz } template -[[nodiscard]] constexpr T AlignDown(T value, std::size_t size) { - static_assert(std::is_unsigned_v, "T must be an unsigned value."); +requires std::is_unsigned_v[[nodiscard]] constexpr T AlignDown(T value, size_t size) { return static_cast(value - value % size); } template -[[nodiscard]] constexpr bool Is4KBAligned(T value) { - static_assert(std::is_unsigned_v, "T must be an unsigned value."); +requires std::is_unsigned_v[[nodiscard]] constexpr bool Is4KBAligned(T value) { return (value & 0xFFF) == 0; } template -[[nodiscard]] constexpr bool IsWordAligned(T value) { - static_assert(std::is_unsigned_v, "T must be an unsigned value."); +requires std::is_unsigned_v[[nodiscard]] constexpr bool IsWordAligned(T value) { return (value & 0b11) == 0; } template -[[nodiscard]] constexpr bool IsAligned(T value, std::size_t alignment) { - using U = typename std::make_unsigned::type; +requires std::is_integral_v[[nodiscard]] constexpr bool IsAligned(T value, size_t alignment) { + using U = typename std::make_unsigned_t; const U mask = static_cast(alignment - 1); return (value & mask) == 0; } -template +template class AlignmentAllocator { public: using value_type = T; - using size_type = std::size_t; - using difference_type = std::ptrdiff_t; + using size_type = size_t; + using difference_type = ptrdiff_t; using propagate_on_container_copy_assignment = std::true_type; using propagate_on_container_move_assignment = std::true_type; -- cgit v1.2.3