[libcxx-commits] [PATCH] D153211: [libc++][Modules] Make module exports consistent with header includes

Ian Anderson via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon Jun 26 21:44:43 PDT 2023


iana added inline comments.


================
Comment at: libcxx/include/cwchar:113
 #include <cwctype>
+#include <stdio.h>
 
----------------
Mordante wrote:
> Is this really needed, it looks weird; C headers should be independent.
I think it was transitively included, but cwchar explicitly exports the module, so it needs to include the header.


================
Comment at: libcxx/include/string_view:230
 #include <__type_traits/type_identity.h>
+#include <initializer_list>
 #include <iosfwd>
----------------
Mordante wrote:
> The header does not use `initializer_list` why is it needed?
string_view's module explicitly exports it, and this one I'm pretty sure wasn't being transitively included. So this export would definitely break without the include being added.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153211



More information about the libcxx-commits mailing list