[libcxx-commits] [PATCH] D107671: [libcxx][ranges] Add `ranges::join_view`.

Tim Song via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Fri Aug 6 21:31:26 PDT 2021


tcanens added inline comments.


================
Comment at: libcxx/include/__ranges/non_propagating_cache.h:89-95
+    template<class _Other>
+    _LIBCPP_HIDE_FROM_ABI
+    constexpr _Tp& __emplace_deref(const _Other& __value) {
+      __value_.reset();
+      __value_.emplace(*__value);
+      return *__value_;
+    }
----------------
This isn't right. See https://github.com/microsoft/STL/pull/2038#discussion_r683086306


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107671



More information about the libcxx-commits mailing list