[libcxx-commits] [PATCH] D119489: [libc++][ranges] Implement `std::mergeable`.

[Louis Dionne via Phabricator via libcxx-commits]

ldionne accepted this revision.
ldionne added a comment.
This revision is now accepted and ready to land.

LGTM with ultra nit.



================
Comment at: libcxx/include/iterator:159-162
+ // [alg.req.mergeable], concept mergeable
+template<class I1, class I2, class Out,
+    class R = ranges::less, class P1 = identity, class P2 = identity>
+  concept mergeable = see below;                           // since C++20
----------------
This is a nit, but can we match the order of the synopsis in the Standard: `indirectly_swappable`, `indirectly_comparable`, `permutable`, `mergeable`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119489