[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