EricWF added a comment. FYI I have a full implementation of this laying around as https://reviews.llvm.org/D27402 (https://reviews.llvm.org/D27402). But I have never taken the time to resolve merge conflicts. Feel free to steal any of the tests if they're still relevant.
================ Comment at: include/experimental/memory_resource:429 + size_t __capacity_; + size_t __alignment_; + size_t __used_; ---------------- I can't imagine we'll need more than 1 byte to represent the alignment. ================ Comment at: include/experimental/memory_resource:474 +protected: + void* do_allocate(size_t __bytes, size_t __alignment); + ---------------- Lets add `override` to these functions. ================ Comment at: src/experimental/memory_resource.cpp:217 +{ + if (void *result = try_allocate_from_chunk(&__original_, bytes, align)) { + return result; ---------------- Drop the braces for conditionals and loops with single statements. Repository: rCXX libc++ https://reviews.llvm.org/D47111 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits