[libcxx-commits] [PATCH] D103327: [libc++] NFC: Move std::copy to its own header

[Christopher Di Bella via Phabricator via libcxx-commits]

cjdb added a comment.

In D103327#2787471 <https://reviews.llvm.org/D103327#2787471>, @Quuxplusone wrote:

> I'd still like to see the long-term plan here: Would you have `__algorithm/ranges_copy.h` sitting alongside `__algorithm/copy.h`? Or would you roll them together into one header? (The former would proliferate headers twice as fast.

We're planning to include `std` and `ranges` algorithms in the same header. That's why there's no `__iterator/ranges_advance.h`.

> But the latter would force all libc++ users of `std::copy` to depend on large swaths of `<ranges>`, so the former is the lesser evil.)

This is objectively false. Users can only obtain `std::copy` by including `<algorithm>`, which imports "large swaths of `<ranges>`" in both cases.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103327