[libcxx-commits] [libcxx] [libc++] P3016R6: Resolve inconsistencies in begin/end for `valarray` and braced initializer lists (PR #173637)

[A. Jiang via libcxx-commits]

================
@@ -272,6 +272,7 @@ namespace std {
 #  include <__functional/unary_function.h>
 #  include <__fwd/functional.h>
 #  include <__iterator/bounded_iter.h>
+#  include <__iterator/reverse_iterator.h>
----------------
frederick-vs-ja wrote:

When I added `<optional>` and `<valarray>` to `‎libcxx/test/std/iterators/iterator.range/mandatory_inclusions.gen.py`, this became required by Clang module builds (probably due to transitive inclusion via `<__iterator/reverse_access.h>`). Do you have a better way for this?

https://github.com/llvm/llvm-project/pull/173637