[libcxx-commits] [PATCH] D100682: [libc++][nfc] Move incrementable_traits and indirectly_readable_traits into separate headers.

[Zoe Carver via Phabricator via libcxx-commits]

zoecarver added inline comments.


================
Comment at: libcxx/include/__iterator/readable_traits.h:16
+
+#if !defined(_LIBCPP_HAS_NO_RANGES)
+
----------------
ldionne wrote:
> It would make sense to hoist that out of this file and into `<iterator>`. Same for incrementable traits.
Eh, I don't know. I'd be OK doing that and adding an `#error` here if it's not defined. I'd kind of like all our headers to always be includable, though. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100682