[libcxx-commits] [PATCH] D102121: [libcxx][ranges] adds _`non-propagating-cache`_
Mark de Wever via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Mon May 10 09:44:24 PDT 2021
Mordante added a comment.
In D102121#2746815 <https://reviews.llvm.org/D102121#2746815>, @cjdb wrote:
>> Can you update the status papers documentation and add a warning based on which version of the paper this feature has been implemented? If the wording changes before the paper is accepted we need to update our implementation to match the final wording.
>
> Done for D102119 <https://reviews.llvm.org/D102119>. I'm not sure if it's necessary to do it for this patch since the cache is an implementation detail and has always been necessary for C++20 ranges. Tying it to P2328 <https://reviews.llvm.org/P2328> implies that it's a C++23 addition, which isn't the case here. Regardless of what the committee decides to do with P2328 <https://reviews.llvm.org/P2328>, we //need// to keep this type.
As discussed on Discord. Since this only adds the exposition only part of the paper I think it's not too important.
Thanks for adding the paper to D102119 <https://reviews.llvm.org/D102119>.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D102121/new/
https://reviews.llvm.org/D102121
More information about the libcxx-commits
mailing list