From 723edb4c06595587d4efed88dd06e69f42183b1e Mon Sep 17 00:00:00 2001 From: Lioncash Date: Fri, 17 Jul 2020 07:26:59 -0400 Subject: alignment: Simplify AlignmentAllocator implementation With C++20, much of the allocator interface has been simplified, so we can make the same adjustments. --- src/common/alignment.h | 47 ++++------------------------------------------- 1 file changed, 4 insertions(+), 43 deletions(-) (limited to 'src/common') diff --git a/src/common/alignment.h b/src/common/alignment.h index b37044bb6..4025ba651 100644 --- a/src/common/alignment.h +++ b/src/common/alignment.h @@ -3,7 +3,6 @@ #pragma once #include -#include #include namespace Common { @@ -54,66 +53,28 @@ public: using size_type = std::size_t; using difference_type = std::ptrdiff_t; - using pointer = T*; - using const_pointer = const T*; - - using reference = T&; - using const_reference = const T&; - using propagate_on_container_copy_assignment = std::true_type; using propagate_on_container_move_assignment = std::true_type; using propagate_on_container_swap = std::true_type; using is_always_equal = std::true_type; -public: constexpr AlignmentAllocator() noexcept = default; template constexpr AlignmentAllocator(const AlignmentAllocator&) noexcept {} - pointer address(reference r) noexcept { - return std::addressof(r); - } - - const_pointer address(const_reference r) const noexcept { - return std::addressof(r); - } - - pointer allocate(size_type n) { - return static_cast(::operator new (n, std::align_val_t{Align})); - } - - void deallocate(pointer p, size_type) { - ::operator delete (p, std::align_val_t{Align}); + T* allocate(size_type n) { + return static_cast(::operator new (n * sizeof(T), std::align_val_t{Align})); } - void construct(pointer p, const value_type& wert) { - new (p) value_type(wert); - } - - void destroy(pointer p) { - p->~value_type(); - } - - size_type max_size() const noexcept { - return size_type(-1) / sizeof(value_type); + void deallocate(T* p, size_type n) { + ::operator delete (p, n * sizeof(T), std::align_val_t{Align}); } template struct rebind { using other = AlignmentAllocator; }; - - bool operator!=(const AlignmentAllocator& other) const noexcept { - return !(*this == other); - } - - // Returns true if and only if storage allocated from *this - // can be deallocated from other, and vice versa. - // Always returns true for stateless allocators. - bool operator==(const AlignmentAllocator& other) const noexcept { - return true; - } }; } // namespace Common -- cgit v1.2.3