[libcxx-commits] [PATCH] D102564: [libcxx][iterator] adds `std::ranges::prev`

[Christopher Di Bella via Phabricator via libcxx-commits]

cjdb updated this revision to Diff 347843.
cjdb added a comment.

Applies @ldionne's feedback (from D102563 <https://reviews.llvm.org/D102563>, since these two patches are siblings):

- removes `[[nodiscard]]` from `ranges::next` (per offline discussion: things not explicitly marked `[[nodiscard]]` in the wording to be on hold till after P2351R0 and P2377R0 are discussed in Library Evolution)
- removes `[[nodiscard]]` from tests
- splits single `next.pass.cpp` into four files, one for each overload


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102564

Files:
  libcxx/include/CMakeLists.txt
  libcxx/include/__iterator/prev.h
  libcxx/include/iterator
  libcxx/test/std/iterators/iterator.primitives/range.iter.ops/range.iter.ops.prev/check_round_trip.h
  libcxx/test/std/iterators/iterator.primitives/range.iter.ops/range.iter.ops.prev/constraints.verify.cpp
  libcxx/test/std/iterators/iterator.primitives/range.iter.ops/range.iter.ops.prev/iterator.pass.cpp
  libcxx/test/std/iterators/iterator.primitives/range.iter.ops/range.iter.ops.prev/iterator_count.pass.cpp
  libcxx/test/std/iterators/iterator.primitives/range.iter.ops/range.iter.ops.prev/iterator_count_sentinel.pass.cpp
  libcxx/test/std/iterators/iterator.primitives/range.iter.ops/range.iter.ops.prev/special_function.compile.pass.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D102564.347843.patch
Type: text/x-patch
Size: 15078 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20210526/f46af458/attachment.bin>