[libcxx-commits] [PATCH] D115686: [libc++] [P0849R8] Introduce _LIBCPP_AUTO_CAST(x) for auto(x)

[Arthur O'Dwyer via Phabricator via libcxx-commits]

Quuxplusone added inline comments.


================
Comment at: libcxx/include/__ranges/access.h:111
       typename iterator_t<_Tp>;
-      { _VSTD::__decay_copy(_VSTD::forward<_Tp>(__t).end()) } -> sentinel_for<iterator_t<_Tp> >;
     };
----------------
ldionne wrote:
> `__t` is "an lvalue that denotes the reified object for `E`", and that's why we don't want to use `std::forward` -- right?
Correct.

Re adding a status-page entry for [P0849](https://wg21.link/p0849r8): Done! Of course, to mark it "complete," I had to expand the scope of this patch a tiny bit... :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115686