[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