[libcxx-commits] [PATCH] D102639: [libcxx][ranges] Add `indirectly_­movable` and `indirectly_movable_storable`.

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon Jun 21 10:00:17 PDT 2021


ldionne accepted this revision.
ldionne added a comment.

LGTM once CI passes, and if you merge `PointerToWithValueType` and `PointerTo` as we discussed offline just now.



================
Comment at: libcxx/test/std/iterators/iterator.requirements/alg.req.ind.move/indirectly_movable.compile.pass.cpp:59
+static_assert( std::indirectly_movable<PointerTo<MoveOnlyWrapper>, PointerTo<MoveOnlyWrapper>>);
+static_assert( std::indirectly_movable<MoveOnly*, PointerTo<MoveOnlyWrapper>>);
----------------
I think you mean `PointerTo<MoveOnly>` instead of `PointerTo<MoveOnlyWrapper>` here (and elsewhere).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D102639/new/

https://reviews.llvm.org/D102639



More information about the libcxx-commits mailing list