Use std::uintptr_t in alignment calculations

This commit is contained in:
Smertig 2021-05-09 14:06:40 +03:00 committed by The Phantom Derpstorm
parent 35c71f1b47
commit 4eac7a02c0

View File

@ -78,42 +78,42 @@ namespace sol {
#endif #endif
} }
inline void* align(std::size_t alignment, std::size_t size, void*& ptr, std::size_t& space, std::size_t& required_space) { inline std::uintptr_t align(std::size_t alignment, std::size_t size, std::uintptr_t ptr, std::size_t& space, std::size_t& required_space) {
// this handels arbitrary alignments... // this handles arbitrary alignments...
// make this into a power-of-2-only? // make this into a power-of-2-only?
// actually can't: this is a C++14-compatible framework, // actually can't: this is a C++14-compatible framework,
// power of 2 alignment is C++17 // power of 2 alignment is C++17
std::uintptr_t initial = reinterpret_cast<std::uintptr_t>(ptr); std::uintptr_t initial = ptr;
std::uintptr_t offby = static_cast<std::uintptr_t>(initial % alignment); std::uintptr_t offby = static_cast<std::uintptr_t>(initial % alignment);
std::uintptr_t padding = (alignment - offby) % alignment; std::uintptr_t padding = (alignment - offby) % alignment;
required_space += size + padding; required_space += size + padding;
if (space < required_space) { if (space < required_space) {
return nullptr; return 0;
} }
ptr = static_cast<void*>(static_cast<char*>(ptr) + padding); ptr += padding;
space -= padding; space -= padding;
return ptr; return ptr;
} }
inline void* align(std::size_t alignment, std::size_t size, void*& ptr, std::size_t& space) { inline std::uintptr_t align(std::size_t alignment, std::size_t size, std::uintptr_t ptr, std::size_t& space) {
std::size_t required_space = 0; std::size_t required_space = 0;
return align(alignment, size, ptr, space, required_space); return align(alignment, size, ptr, space, required_space);
} }
inline void align_one(std::size_t a, std::size_t s, void*& target_alignment) { inline void* align(std::size_t alignment, std::size_t size, void* ptr, std::size_t& space) {
return reinterpret_cast<void*>(align(alignment, size, reinterpret_cast<std::uintptr_t>(ptr), space));
}
inline std::uintptr_t align_one(std::size_t a, std::size_t s, std::uintptr_t target_alignment) {
std::size_t space = (std::numeric_limits<std::size_t>::max)(); std::size_t space = (std::numeric_limits<std::size_t>::max)();
target_alignment = align(a, s, target_alignment, space); return align(a, s, target_alignment, space) + s;
target_alignment = static_cast<void*>(static_cast<char*>(target_alignment) + s);
} }
template <typename... Args> template <typename... Args>
std::size_t aligned_space_for(void* alignment = nullptr) { std::size_t aligned_space_for(std::uintptr_t alignment) {
// use temporary storage to prevent strict UB shenanigans std::uintptr_t start = alignment;
char alignment_shim[(std::max)({ sizeof(Args)... }) + (std::max)({ alignof(Args)... })] {}; (void)detail::swallow { int {}, (alignment = align_one(std::alignment_of_v<Args>, sizeof(Args), alignment), int {})... };
char* start = alignment != nullptr ? static_cast<char*>(alignment) : alignment_shim; return static_cast<std::size_t>(alignment - start);
alignment = start;
(void)detail::swallow { int {}, (align_one(std::alignment_of_v<Args>, sizeof(Args), alignment), int {})... };
return static_cast<std::size_t>(static_cast<char*>(alignment) - start);
} }
inline void* align_usertype_pointer(void* ptr) { inline void* align_usertype_pointer(void* ptr) {
@ -229,8 +229,8 @@ namespace sol {
T** pointerpointer = static_cast<T**>(alloc_newuserdata(L, sizeof(T*))); T** pointerpointer = static_cast<T**>(alloc_newuserdata(L, sizeof(T*)));
return pointerpointer; return pointerpointer;
} }
static const std::size_t initial_size = aligned_space_for<T*>(nullptr); static const std::size_t initial_size = aligned_space_for<T*>(0x0);
static const std::size_t misaligned_size = aligned_space_for<T*>(reinterpret_cast<void*>(0x1)); static const std::size_t misaligned_size = aligned_space_for<T*>(0x1);
std::size_t allocated_size = initial_size; std::size_t allocated_size = initial_size;
void* unadjusted = alloc_newuserdata(L, initial_size); void* unadjusted = alloc_newuserdata(L, initial_size);
@ -341,8 +341,8 @@ namespace sol {
compilers are optimized for aligned reads/writes compilers are optimized for aligned reads/writes
(and clang will barf UBsan errors on us for not being aligned) (and clang will barf UBsan errors on us for not being aligned)
*/ */
static const std::size_t initial_size = aligned_space_for<T*, T>(nullptr); static const std::size_t initial_size = aligned_space_for<T*, T>(0x0);
static const std::size_t misaligned_size = aligned_space_for<T*, T>(reinterpret_cast<void*>(0x1)); static const std::size_t misaligned_size = aligned_space_for<T*, T>(0x1);
void* pointer_adjusted; void* pointer_adjusted;
void* data_adjusted; void* data_adjusted;
@ -392,8 +392,8 @@ namespace sol {
return mem; return mem;
} }
static const std::size_t initial_size = aligned_space_for<T*, unique_destructor, unique_tag, Real>(nullptr); static const std::size_t initial_size = aligned_space_for<T*, unique_destructor, unique_tag, Real>(0x0);
static const std::size_t misaligned_size = aligned_space_for<T*, unique_destructor, unique_tag, Real>(reinterpret_cast<void*>(0x1)); static const std::size_t misaligned_size = aligned_space_for<T*, unique_destructor, unique_tag, Real>(0x1);
void* pointer_adjusted; void* pointer_adjusted;
void* dx_adjusted; void* dx_adjusted;
@ -462,8 +462,8 @@ namespace sol {
return pointer; return pointer;
} }
static const std::size_t initial_size = aligned_space_for<T>(nullptr); static const std::size_t initial_size = aligned_space_for<T>(0x0);
static const std::size_t misaligned_size = aligned_space_for<T>(reinterpret_cast<void*>(0x1)); static const std::size_t misaligned_size = aligned_space_for<T>(0x1);
std::size_t allocated_size = initial_size; std::size_t allocated_size = initial_size;
void* unadjusted = alloc_newuserdata(L, allocated_size); void* unadjusted = alloc_newuserdata(L, allocated_size);