[libcxx-commits] [PATCH] D127675: [libc++] Removes unneeded <iterator> includes.

[Mark de Wever via Phabricator via libcxx-commits]

Mordante added a comment.

In D127675#3587954 <https://reviews.llvm.org/D127675#3587954>, @sberg wrote:

> But now
>
>   #include <array>
>   auto b = std::begin("");
>
> fails to compile, even though [iterator.range]/1 states:  "In addition to being available via inclusion of the <iterator> header, the function templates in 23.7 are
> available when any of the following headers are included: <array> (22.3.2), [...]"

Thanks for the comment! @philnik created D127953 <https://reviews.llvm.org/D127953> which addresses this issue.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127675