[libcxx-commits] [PATCH] D90999: [libc++] Implements ranges::enable_borrowed_range

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon Nov 23 14:42:07 PST 2020


ldionne added a comment.

In D90999#2381043 <https://reviews.llvm.org/D90999#2381043>, @miscco wrote:

> I am wondering whether we should split the ranges header up into a smaller part that we use to include in other headers adn the rest.
>
> Including all of ranges just for one trait seems a bit of an overkill.
>
> We could always add it to concepts as that should be a transitive include for everything that includes range. Thoughts @ldionne?

I'd be fine with splitting various parts of ranges into separate headers if that makes the implementation clearer, etc.


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

https://reviews.llvm.org/D90999



More information about the libcxx-commits mailing list