[libcxx-commits] [PATCH] D136268: [libc++][ranges] implement `std::views::elements_view`

[Hui via Phabricator via libcxx-commits]

huixie90 added inline comments.


================
Comment at: libcxx/include/__iterator/concepts/can_reference.h:2
+// -*- C++ -*-
+//===----------------------------------------------------------------------===//
+//
----------------
philnik wrote:
> I don't think we should granularize `__iterator/concepts.h`. Compiling the header takes just ~35ms, 20 of which are including `<type_traits>`. So we'd save maybe 10ms by granularizing it, which I don't think is worth it. `__iterator/concepts.h` also isn't very large at ~300 LoC, so it also doesn't increase readability much.
I wasn't too concerned about compilation time. Without doing this, we will add public dependency from `<tuple>` to `<concepts>` (see description). I'd like to avoid adding dependencies like this so lifted several headers required by `subrange`'s forward declaration


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136268