[PATCH] D136242: [IntervalMap] Add move and copy ctors and assignment operators
David Blaikie via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 19 10:29:22 PDT 2022
dblaikie accepted this revision.
dblaikie added a comment.
This revision is now accepted and ready to land.
Sounds good
================
Comment at: llvm/include/llvm/ADT/IntervalMap.h:1044
public:
- explicit IntervalMap(Allocator &a) : height(0), rootSize(0), allocator(a) {
+ explicit IntervalMap(Allocator &a) : height(0), rootSize(0), allocator(&a) {
assert((uintptr_t(&data) & (alignof(RootLeaf) - 1)) == 0 &&
----------------
could drop the `height` and `rootSize` initializations now that they're initialized at the member declaration
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D136242/new/
https://reviews.llvm.org/D136242
More information about the llvm-commits
mailing list