[libcxx-commits] [PATCH] D102037: [libcxx][views] Add drop_view.

[Christopher Di Bella via Phabricator via libcxx-commits]

cjdb requested changes to this revision.
cjdb added inline comments.
This revision now requires changes to proceed.


================
Comment at: libcxx/include/__ranges/drop_view.h:44
+    range_difference_t<_View> __count = 0;
+    iterator_t<_View> __cached_begin = ranges::begin(__base);
+
----------------
Caching should only be available for forward ranges (it's a logic error to cache for input ranges, and also not to spec). Also, please make this `[[no_unique_address]]`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102037