[libcxx-commits] [PATCH] D101193: [libcxx][ranges] Add ranges::empty CPO.

[Zoe Carver via Phabricator via libcxx-commits]

zoecarver added a comment.

Thanks for the review, Arthur. Will update shortly.



================
Comment at: libcxx/test/std/ranges/range.access/range.prim/empty.pass.cpp:89
+};
+static_assert(!std::is_invocable_v<RangeSizeT, BeginEndNotSizedSentinel&>);
+
----------------
Quuxplusone wrote:
> If this was cut-and-pasted from the `ranges::size` tests... should this now say `RangeEmptyT`?
This (and below) is intentional. I wanted to make sure that this wasn't choosing the `__can_invoke_size` overload.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101193