[libcxx-commits] [PATCH] D103329: [libcxx][nfc] moves std `advance`, `next`, and `prev` into their headers

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon May 31 08:03:38 PDT 2021


ldionne accepted this revision.
ldionne added a comment.
This revision is now accepted and ready to land.

I'm not a big fan of putting `std::advance` and `std::ranges::advance` in the same header - they are related but at the same time, they are fundamentally different things and have different dependencies. That's one reason why I preferred to split `ranges::` things into `<__ranges/XXXX.h>` instead. But I guess this will do, we can adjust in the future if need be.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103329



More information about the libcxx-commits mailing list