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

[Nikolas Klauser via Phabricator via libcxx-commits]

philnik added inline comments.


================
Comment at: libcxx/include/__iterator/concepts/can_reference.h:2
+// -*- C++ -*-
+//===----------------------------------------------------------------------===//
+//
----------------
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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136268