[libcxx-commits] [PATCH] D156177: [libc++][Modules] Recreate the top level `std` clang module
Ian Anderson via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Thu Jul 27 13:42:32 PDT 2023
iana marked 2 inline comments as done.
iana added inline comments.
================
Comment at: libcxx/docs/Contributing.rst:50
+ - Did you add it to ``include/__std``?
- Did you add it to ``include/module.modulemap.in``?
----------------
Mordante wrote:
> iana wrote:
> > Mordante wrote:
> > > Since we're going to add a script, so this can be removed.
> > Discussed in Discord, I'll make a script in a separate review later today, I'll remove this comment then.
> I would like to remove it anyway. We want to backport this patch to LLVM-17 which then contains incorrect information. For main the information will be wrong for a few days; I don't expect new headers in that short time-span.
Done!
================
Comment at: libcxx/utils/libcxx/test/header_information.py:18-21
- "experimental/algorithm": "// UNSUPPORTED: c++03",
"experimental/deque": "// UNSUPPORTED: c++03",
"experimental/forward_list": "// UNSUPPORTED: c++03",
- "experimental/functional": "// UNSUPPORTED: c++03",
----------------
Mordante wrote:
> When the CI is green can you commit this separately. I like to not have this in this patch. Normally I don't mind some additional polishing, but I want to keep the changes for LLVM-17 as small as possible.
Yep!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D156177/new/
https://reviews.llvm.org/D156177
More information about the libcxx-commits
mailing list