[libcxx-commits] [PATCH] D101948: [libc++] Future-proof std::copy for ranges

[Louis Dionne via Phabricator via libcxx-commits]

ldionne added a comment.

In D101948#2740500 <https://reviews.llvm.org/D101948#2740500>, @Quuxplusone wrote:

> You can guess that I dislike any approach that seems to lead to one-file-per-function. :)  What I personally would do is,
> [...]
>
> `std::copy` is also an "interesting" first testbed, because it's all tangled up with `__wrap_iter` and `__is_cpp17_contiguous_iterator`, which are still quite actively mutating at the moment.

Thanks for your comments. It seems that our two approaches are basically the same except when it comes to how we are to split the header files.

I would like to have more discussion around aspects like how we are to bridge between ranges and classic algorithms in the most efficient and simplest way possible.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101948