[Draft][libc++][modules] Adds std module.

[Mordante]

Summary

Implements parts of

• P2465R3 Standard Library Modules std and std.compat

This patch is based on work by @ChuanqiXu.

This patch not intended to be landed is this form, it's intended to get
a feeling of modules in libc++ and improve upon it. This patch will be
used to discuss the further approach regarding modules in libc++.

[EricWF]

Cool deal. Thanks for working on this.

[ldionne]

libcxx/stdmodules/std-format.cppm

12 ↗

(On Diff #502156)

Well, if nobody like synopsis in headers, we can also remove them :-). Let's chat about that.

To be clear, you are not opposed to keeping the synopsis comment in test files that describes what we're testing, right? I think those are definitely useful and we should not stop those.

Pinging @var-const @huixie90 for thoughts. If folks agree, I don't want to impose it on people.

FWIW I do agree that storing those in the .cppm files results in something less readable than I thought it would.

[ldionne]

Pinging @iana @Bigcheese for awareness since they are working on Modules.

[philnik]

libcxx/stdmodules/std-format.cppm

12 ↗

(On Diff #502156)

Well, if nobody like synopsis in headers, we can also remove them :-). Let's chat about that.

To be clear, you are not opposed to keeping the synopsis comment in test files that describes what we're testing, right? I think those are definitely useful and we should not stop those.

I'm not sure they are always super useful, but they are definitely in enough cases that it would probably be a bad idea to remove them. They are also much easier to maintain and less error-prone.

[Mordante]

In D144994#4176269, @EricWF wrote:

Cool deal. Thanks for working on this.

Thanks!

In D144994#4173764, @ChuanqiXu wrote:

One of the issues in libc++ is that vendors can decide to disable filesystem header. Since parts of that library are in the dylib the std module can't provide what's in filesystem. Does your method also have that feature?

I disabled the filesystem header directly in the downstream... so I don't have a lot experience in this.

Thanks for the information!

I think we can discuss that and see whether it's a blocker for landing the patch or not. For now I still have issues with ranges (which I have not started to investigate). I have been experimenting with running the test-suite for the modular build. So I will have some more information on that later today, probably on Discord.

Yeah, it is always good to have more experience. And I didn't expect that we can convert all headers and pass all the tests in one shot. It is good enough for me to have a std module which contains a lot of headers and can pass a lot of tests.

I also didn't expect that, but we need to look at when we're comfortable with shipping something. Since libc++ is a system library on some platforms we need to consider that carefully. Obviously there are other non-official ways to ship things too.

It seems quite some tests fail due to using size_t instead of std::size_t and similar C-types. I'm working on patches to use the qualified name instead.

libcxx/stdmodules/std-format.cppm

12 ↗

(On Diff #502156)

Well, if nobody like synopsis in headers, we can also remove them :-). Let's chat about that.

To be clear, you are not opposed to keeping the synopsis comment in test files that describes what we're testing, right? I think those are definitely useful and we should not stop those.

Pinging @var-const @huixie90 for thoughts. If folks agree, I don't want to impose it on people.

FWIW I do agree that storing those in the .cppm files results in something less readable than I thought it would.

+1 for discussing the synopsis. But let's take it to Discord or a libc++ meeting.

[Mordante]

Various improvements.

• Incorporates D145596 CMake 3.26 fix.
• Removes new tests libc++ certainly doesn't want to duplicate all their tests for modules. (The final solution still is unknown.)
• Adds the std.compat module.
• Propagate some libc++ configuration flags to the modules
• Prepared a CI job, this is not used yet since the CI does not have CMake 3.26 and Ninja 1.11 yet.
• Update the test transformation script and did initial tried it locally.
• Small fixes to the module file.
• Added a few now module files.

[aaronmondal]

libcxx/docs/Modules.rst

84

Ah yes this fails due to template (mis)deduction with lambdas if I remember correctly. This works around it but it was decided some time ago that this shouldn't be changed in libc++ due to "work around compiler bug" not being a good reason for a change.

diff --git a/libcxx/include/__compare/synth_three_way.h b/libcxx/include/__compare/synth_three_way.h
index fa8cbda79ba2..a6df1b936c72 100644
--- a/libcxx/include/__compare/synth_three_way.h
+++ b/libcxx/include/__compare/synth_three_way.h
@@ -25,8 +25,8 @@ _LIBCPP_BEGIN_NAMESPACE_STD
 
 // [expos.only.func]
 
-_LIBCPP_HIDE_FROM_ABI inline constexpr auto __synth_three_way =
-  []<class _Tp, class _Up>(const _Tp& __t, const _Up& __u)
+template<class _Tp, class _Up>
+_LIBCPP_HIDE_FROM_ABI inline constexpr auto __synth_three_way(const _Tp& __t, const _Up& __u)
     requires requires {
       { __t < __u } -> __boolean_testable;
       { __u < __t } -> __boolean_testable;

libcxx/stdmodules/std-string_view.cppm

47 ↗

(On Diff #504447)

[h-vetinari]

libcxx/docs/Modules.rst

66

But we need to find the "proper" location to store them. Since this is something new this probably needs some involvement of our packagers. I don't expect the Linux Standard Base has an entry for C++20 modules.

As mentioned already, I think there are strong parallels with distributing headers. The bare minimum should be that they are automatically discoverable by the toolchain that produced them (should be somewhere under INSTALL_PREFIX obviously), but ideally it's in a standard location that's uniform across toolchains (again, relative to a PREFIX), like the headers and libs are too. I'm not sure if SG21 has had discussions about this, or if CMake has already thought about this.

Personally, I think the best might be a separate folder like $PREFIX/something_module_related/foo.cppm, but I wouldn't care much or at all if it's under $PREFIX/include/something_module_related/foo.cppm or $PREFIX/lib/<libname>/foo.cppm, etc., as long as it's discoverable and works.

CC @mgorny as another packager & @ben.boeckel for the CMake side.

[ChuanqiXu]

libcxx/docs/Modules.rst

66

I'm not sure if SG21 has had discussions about this, or if CMake has already thought about this.

As far as I know, sg15 has discussed this for a long time. (If I recall this correctly,) sg15 don't get the final consensus yet. But @ruoso has already made a lot of proposals. We can look at them later.

Personally, I think the best might be a separate folder like $PREFIX/something_module_related/foo.cppm, but I wouldn't care much or at all if it's under $PREFIX/include/something_module_related/foo.cppm or $PREFIX/lib/<libname>/foo.cppm, etc., as long as it's discoverable and works.

I think we need to discuss this in the discourse. Because the patch is going to be a large patch and the topic about distribution is another big topic. It looks not good to discuss two big things in the same page. The direction may be lost.

libcxx/stdmodules/std-compat.cppm

1 ↗

(On Diff #504447)

std.compat.cppm may be a better name. Since std-compat.cppm looks like a partition of the std module to me.

[ChuanqiXu]

libcxx/docs/Modules.rst

66

I created https://discourse.llvm.org/t/rfc-about-the-default-location-for-std-modules/69191 and let's discuss this there.

libcxx/stdmodules/CMakeLists.txt.in

71–75 ↗

(On Diff #504447)

This is what I did in the downstream. I imaged that we will merge this one with libc++.so in the upstream. But maybe I am wrong.
I feel we need to give it another name like libc++-modules. Otherwise it may be conflict with the std modules provided by libstdc++ later.

[ben.boeckel]

libcxx/CMakeLists.txt

109–112

This help string has embedded newlines and indentation (though I see other options do this…). I don't think the UI editors show the help well in this case (testing with ccmake seems to bear this out).

427

Somewhere under libdir would be good unless we know these files are actually arch-independent.

libcxx/docs/Modules.rst

26–27

Note that buildsystems should handle this. I don't know how viable the "just run the compiler by hand" is in the long-term…

43

You cannot ship the P1689 *output*, but it needs to be generated by anything *consuming* it so that the collation step can know to add the right flags (-fmodule-file=…) to the consuming TU compilation.

52–55

Nor should there be IMO. Note that a single build tree may need *multiple* compilations of the module, so I don't think a single "scanning output" is suitable to ship.

66

Personally, I think the best might be a separate folder like $PREFIX/something_module_related/foo.cppm, but I wouldn't care much or at all if it's under $PREFIX/include/something_module_related/foo.cppm or $PREFIX/lib/<libname>/foo.cppm, etc., as long as it's discoverable and works.

Note that libstdc++ is also going to ship std.cppm (possibly with a different extension), so the libname being part of the path sounds best to me. I'll jump in over on Discourse.

72

"not used as a compile flag" perhaps?

117

IMO, no.

163–165

Is this intended to be a stable interface? IMO, support for std modules needs to come from CMake because this is going to be different for every compiler otherwise.

libcxx/stdmodules/CMakeLists.txt

43 ↗

(On Diff #501265)

Instead we need to focus on distributing .cppm files and offer the tools for the user to create the .pcm files for their project. This is something I will investigate further.

I think this is more the job of CMake since asking users to do

cmake
if (<is_clang>)
  setup_clang_libcpp()
elseif (<is_gcc>)
  setup_gcc_libstdcpp()
elseif (<is_msvc>)
  setup_msvc_stl()
endif ()

is awfully narrow IMO. Highly related question: what do you expect to happen in the case of clang with libstdc++ (as many distros end up doing)?

libcxx/stdmodules/CMakeLists.txt.in

7 ↗

(On Diff #504447)

3.26 should have this.

9 ↗

(On Diff #504447)

This UUID changes…regularly. There will probably need to be a table to do this "properly".

10 ↗

(On Diff #504447)

This is specific to CMake's test suite and shouldn't be relevant.

[ldionne]

libcxx/stdmodules/std-cstddef.cppm

13 ↗

(On Diff #504447)

Would this make sense to avoid users having to define all the config site macros? #include <__config_site>

[tahonermann]

libcxx/stdmodules/std-cstddef.cppm

13 ↗

(On Diff #504447)

Named modules don't export/leak macros when imported, so the addition of the #include would have no effect on importers of the module.

[Mordante]

libcxx/CMakeLists.txt

109–112

Yes I indeed copied this from the other scripts. Are there better ways in CMake to have multi-line options?

427

This is used for the .cppm source files so they are arch independent.

libcxx/docs/Modules.rst

66

I'm not sure if SG21 has had discussions about this, or if CMake has already thought about this.

As far as I know, sg15 has discussed this for a long time. (If I recall this correctly,) sg15 don't get the final consensus yet. But @ruoso has already made a lot of proposals. We can look at them later.

Do you have a paper number?

Personally, I think the best might be a separate folder like $PREFIX/something_module_related/foo.cppm, but I wouldn't care much or at all if it's under $PREFIX/include/something_module_related/foo.cppm or $PREFIX/lib/<libname>/foo.cppm, etc., as long as it's discoverable and works.

I think we need to discuss this in the discourse. Because the patch is going to be a large patch and the topic about distribution is another big topic. It looks not good to discuss two big things in the same page. The direction may be lost.

I agree, this list is mainly added for documentation so we know what the status is. It was not intended to be used for discussions during the review.

163–165

I agree this is not the way users should use it. But without build system support this is how it works now.
So I feel this is "good enough" for early adapters. However when we (libc++) know how we want to distribute it
we should reach out to build system vendors and ask them to add proper support in their tools.

I'm not even sure what direction we want to take. For the experimental library we have a Clang flag which
automatically link that library. Do we want to do something similar here or leave it entirely in the hands
of the build system?

libcxx/stdmodules/CMakeLists.txt

43 ↗

(On Diff #501265)

I agree this is narrow and not great. However this patch's main focus it to be able to get modules working in libc++ in the first place.
We definitely need to think about the distribution in a user-friendly way.

I'm not sure whether libc++ needs to consider the configuration of Clang + libstdc++. Do you have something specific in mind?
I expect there need to be things done on the Clang side.

libcxx/stdmodules/CMakeLists.txt.in

7 ↗

(On Diff #504447)

Thanks for the information! I see it landed recently in CMake.

9 ↗

(On Diff #504447)

Is there a public available table for these UUIDs? The last time I needed the one for CMake 3.25 I looked in the CMake git history.

libcxx/stdmodules/std-cstddef.cppm

13 ↗

(On Diff #504447)

The goal is not to leak the macros, but to use the macros to conditionally export symbols. For example in the next file in the patch I use #ifndef _LIBCPP_HAS_NO_WIDE_CHARACTERS to avoid exporting wide character functions when the user configured libc++ without wide character support.

When building the BMI we need to know the configuration of libc++. In a similar fashion we need to disable symbols based on the language version used. For now there are some examples for C++20 vs C++23. Not that we have decided that we want to back port this feature, but we will need this for C++23 vs C++26.

[ben.boeckel]

libcxx/docs/Modules.rst

163–165

I think that some build-system agnostic data (similar to the MSVC STL JSON file linked to on Discourse is best). I don't think every compiler trying to chase umpteen build tool integrations (as compilers tend to be *far* harder to upgrade in the Real World than build system tools IME).

ISO C++'s SG15 is the best place for communication IMO.

libcxx/stdmodules/CMakeLists.txt.in

9 ↗

(On Diff #504447)

The UUID is only documented in the CMake repository's Help/dev/experimental.rst file.

[Mordante]

Adds more modules.

This mostely completes the modules. These modules have been compile
tested but nothing more. They also do not take disabled libc++ parts
into account.

Updates the documentation.

[ldionne]

libcxx/test/libcxx/language.support/support.types/cstddef.compile.pass.cpp

1

IMO it would make sense to split this test like this:

/////////////////////////////////////////////////////////////////////////////
// cstddef.compile.pass.cpp
#include <cstddef>
#include "test_macros.h"

using PtrdiffT = ::ptrdiff_t;
using SizeT = ::size_t;
#if TEST_STD_VER >= 11
using MaxAlignT = ::max_align_t;
#endif

// Supported in C++03 mode too for backwards compatibility with previous versions of libc++
using NullptrT = ::nullptr_t;

// Also ensure that we provide std::nullptr_t in C++03 mode, which is an extension too.
using StdNullptrT = std::nullptr_t;
/////////////////////////////////////////////////////////////////////////////

/////////////////////////////////////////////////////////////////////////////
// cstddef.std.module.verify.cpp
// REQUIRES: modules-build (????)
import std;

#include "test_macros.h"

using PtrdiffT = ::ptrdiff_t; // expected-error {{no ptrdiff_t in global namespace}}
using SizeT = ::size_t; // expected-error {{no size_t in global namespace}}

[ etc... ]
/////////////////////////////////////////////////////////////////////////////

/////////////////////////////////////////////////////////////////////////////
// cstddef.stdcompat.module.compile.pass.cpp
import std.compat;

#include "test_macros.h"

using PtrdiffT = ::ptrdiff_t;
using SizeT = ::size_t;

[ etc... ]
/////////////////////////////////////////////////////////////////////////////

This one would be split up manually since it is extremely relevant to test this with C++23 modules.

However, many other tests are not as useful to test with/without modules. As we discussed, we could perhaps run use_modules_in_test.py on the fly to create a modularized version of the test that we wouldn't check into source control.

[Mordante]

Minor improvements

Ran the tests using the compat module with the tests converted using
the test conversion script. After some minor fixes 222 tests fail and
need triaging. (This are more tests than before, but now the tests use
less headers since most headers are now available as module.)

• Improved the conversion script
  • Imports are moved the the end.
  • Modularized header #ifndefs are grouped.
  • Disables tests using deprecated features. These use a compiler switch that doesn't work with modules.
• Updated the documentation.
• Minor module fixes.

[Mordante]

Updated due to Discourse discussion and minor improvements.

• Use the installation location as proposed on Discourse. This adds a proof-of-concept installer.
• Add CI jobs to test modules with parts disabled.
• Fix modules to work with no localization or no wide characters. Other configurations are errors.
• Adds a few empty modules, their headers export nothing.
• Improve the algorithm module.

[tschuett]

libcxx/docs/Modules.rst

105

C++20

137

I believe this is confusing. I thought that you want to store everything underneath c++? Additionally you use the same path for includes and modules.

[aaronmondal]

libcxx/docs/Modules.rst

137

Typo? Should probably be "Includes are stored in <prefix>/include/c++/v1, modules in <prefix>/modules/c++/v1".

I think this makes sense. Modules can't be #included anyways so there is a need for them to live in an include/ directory. And for those using a different ABIs this leaves the possibility to have

.../include/c++/v1
.../include/c++/v2
.../modules/c++/v1
.../modules/c++/v2

or similar to clearly differentiate the headers/module interfaces.

[tschuett]

There was objection against using /usr/modules on discourse. I assumed that libc++ owns /usr/include/c++ and hides everything under there.

[ruoso]

libcxx/docs/Modules.rst

137

$PREFIX/modules is a new path that doesn't exist in the Filesystem Hierarchy Standard.

The source files for the module units is indeed a arch-independent resource, therefore the correct directory would be something under $PREFIX/share

If we ever intend to ship BMI files, they would belong in $libdir.

Here's how it would look like if we go in that direction:

$PREFIX/
   $libdir/
       libc++.so
       libc++.so.module-info
       c++/
           modules/
                libc++/
                      std.gcm.deadbeef1234
   share/
       c++/
           modules/
                 libc++/
                       std.cppm

the libc++.so.module-info file would have the metadata necessary for someone to understand how to produce their own BMI as well as potentially reuse the shipped BMI if it just so happen that they can.

[ruoso]

libcxx/docs/Modules.rst

137

There is one important bit that worries about splitting off the source files, tho.

The module metadata shipped alongside the library itself needs to reference those source files, and I am concerned that requiring the use of ../../../../share/ in order to address the source location can lead to fragility in the deployment.

So, even though it's technically arch-independent, I would also consider the following option:

$PREFIX/
   $libdir/
       libc++.so
       libc++.so.module-info
       c++/
           modules/
                libc++/
                      std.gcm.deadbeef1234
                      std.cppm

Because at that point, the metadata could reference a relative path from the library location without the awkwardness that using ../ can cause when directories are symlinks.

[ChuanqiXu]

Add CI jobs to test modules with parts disabled.

So, is the CI going to be ready?

libcxx/modules/std.compat/cassert.cppm

11–17

What's the meaning for the file? It looks useless.

[Mordante]

In D144994#4208434, @ChuanqiXu wrote:

Add CI jobs to test modules with parts disabled.

So, is the CI going to be ready?

Not at the moment. There are some CI issues not related to modules which should be resolved first. However I am testing with the CI configuration locally. I'm still fixing bugs in the modules and investigating issue which might be Clang bugs.

I'm also still busy to remove symbols from the global namespace so I can start to test the std module, for now I only test with the std.compat module.

libcxx/docs/Modules.rst

26–27

I fully agree. For now we will also ship a CMake file to aid users. But once build systems catch up that CMake file will be removed. At the moment we're not even sure where our .cppm files will be installed, so asking buildsystems to support libc++ is a bit premature. Having this information makes it possible for early adaptors to get started.

43

I've removed the section.

137

This is indeed a typo.

163–165

I will consider attending in SG15 to discuss this.

libcxx/modules/std.compat/cassert.cppm

11–17

It kind of is, but it's easier to have tooling that makes sure every header is a module. This way there need to be no exceptions. Some of the "empty" modules may get symbols in the future.

[ChuanqiXu]

Not at the moment. There are some CI issues not related to modules which should be resolved first. However I am testing with the CI configuration locally. I'm still fixing bugs in the modules and investigating issue which might be Clang bugs.

Oh, got it. Never be hurry : )

libcxx/modules/std.compat/cassert.cppm

11–17

Got your point. I remember the paper said clearly that assert and errno is not exported in the std modules. But it doesn't matter really.

[Mordante]

libcxx/modules/std.compat/cassert.cppm

11–17

I can't find that in the paper nor in the WP. There is http://eel.is/c++draft/std.modules#note-2 which states

[Note 2: Like all named modules, the C++ library modules do not make macros visible ([module.import]), such as assert ([cassert.syn]), errno ([cerrno.syn]), offsetof ([cstddef.syn]), and va_arg ([cstdarg.syn]).
— end note]

[ChuanqiXu]

libcxx/modules/std.compat/cassert.cppm

11–17

Oh, my bad for confusing : )

[Mordante]

Several minor improvements.

• updates comments
• removes C++20 modules
• adds new module specific tests using AST matchers
• fixes several module bugs, mainly found by the new tests

[Mordante]

Rebased on main and resolved merge conflicts.

[ChuanqiXu]

libcxx/test/libcxx/module_std.sh.cpp

228

I can't search libcpp-header-exportable-declarations. Is there a patch that haven't been uploaded?

[Mordante]

libcxx/test/libcxx/module_std.sh.cpp

228

It's in the file libcxx/test/tools/clang_tidy_checks/header_exportable_declarations.cpp which is part of this patch.
(It still has some issues, but I wanted to post some progress again.)

[philnik]

libcxx/test/tools/clang_tidy_checks/header_exportable_declarations.cpp

93

103–107

I think that's the idiomatic way to do this.

142–157

We have something very similar in ugilfy_attributes.cpp. Maybe we should add a header for a few utilities?

166

Is this equivalent to decl->getName()?

173

What is this doing?

libcxx/test/tools/clang_tidy_checks/header_exportable_declarations.hpp

22

Would an unordered_set be better?

[Mordante]

Thanks for the review!

libcxx/test/tools/clang_tidy_checks/header_exportable_declarations.cpp

93

Thanks I already spotted this locally.

103–107

Interesting I wasn't aware llvm::isa multiple arguments. For now I keep it as is, but leave this comment open,

Once the code is working as intended I will see whether this change improves the code.

142–157

That might be useful. BTW if (str[0] == '_' && str[1] >= 'A' && str[1] <= 'Z' does not work properly in EBCDIC.

166

The documentation in not fully clear, but since getName() returns a StringRef I'm sure it's not the same.
getName() would return fill where get_qualified_name() returns std::fill or std::ranges::fill.

173

Dumping a line like using std::vector. This checker is not a real checker, but extracts information.
The output of a toplevel header is compared with the output of a module.
This is done in libcxx/test/libcxx/module_std.sh.cpp (which very much under review and I'm still looking at the failures.)

libcxx/test/tools/clang_tidy_checks/header_exportable_declarations.hpp

22

I haven't tested. Most headers don't have a lot of declarations. The module type_traits which is one of the larger modules has less than 300 entries. (In total there are less than 4000 entries in the Standard library.)

[ChuanqiXu]

libcxx/modules/CMakeLists.txt.in

49

I guess we need a better library name before landing this.

[Mordante]

libcxx/modules/CMakeLists.txt.in

49

What would you propose?
I'm not even sure we really need to, but this is something to determine during a review.

[ChuanqiXu]

libcxx/docs/Modules.rst

84–99

Will these problems be blockers actually? Specially for the "header-module" order problem, we can't fix it fundamentally. And I feel 'std module' is still workable in some degree with these defects. If these problems are not blockers actually, could we move the "limitations" section?

libcxx/modules/CMakeLists.txt.in

49

What would you propose?

I use libstdmodule.a in the downstream. But it looks not good enough for the upstream.

I'm not even sure we really need to, but this is something to determine during a review.

Ideally this should be handled by cmake (or other build systems). But we need it now to make progress. How about something like 'libc++_std_module.a'?

[Mordante]

libcxx/docs/Modules.rst

84–99

I think this one isn't I think we should downgrade this one to limitation.

[aaronmondal]

Any idea how to get this diff from phabricator in a way that doesn't include the changes to the modules directory? That part somehow messes up Bazel's patching.

libcxx/modules/CMakeLists.txt.in

49

Don't have a strong opinion on this, though this will probably be painful to change once users adopt any name.

libc++-std.a, lib++_std.a? Though I personally think just std would be fine.

The std.compat archive should follow a similar scheme though.

[ChuanqiXu]

libcxx/docs/Modules.rst

84–99

What do you mean for this one?

[aaronmondal]

Started testing the Bazel build. Seems to almost build. The __new is a slight inconvenience, but easily worked around and I don't think we can do too much about it.

Only std:memory doesn't build yet on my end.

I'm also noticing that this will place quite the significant restriction on the compiler building the modules. It's a bit unintuitive to build libc++ with C++20 and the interfaces/partitions with C++23. It's more intuitive to build the entire libc++ (sources) with C++23 (which works), but I'd assume that the only compiler capable of doing that at the moment upstream Clang itself 🤣

libcxx/modules/CMakeLists.txt.in

49

I think I'm misunderstanding this target. What are the contents of this archive?

libcxx/modules/std/memory.cppm

184–185

This seems to break the build for me with "no member atomic in namespace std". Where is this symbol coming from?

The <atomic> include is disabled for C++23. Maybe I'm just missing some config flag?

[Mordante]

In D144994#4282770, @aaronmondal wrote:

Started testing the Bazel build. Seems to almost build. The __new is a slight inconvenience, but easily worked around and I don't think we can do too much about it.

Yeah we can't use new since it's a keyword.

I'm also noticing that this will place quite the significant restriction on the compiler building the modules. It's a bit unintuitive to build libc++ with C++20 and the interfaces/partitions with C++23. It's more intuitive to build the entire libc++ (sources) with C++23 (which works), but I'd assume that the only compiler capable of doing that at the moment upstream Clang itself 🤣

We can't nicely build libc++ with C++23, this requires a newer CMake. (I have a patch for it, but I'm waiting for the last buildbots to be updated.)
It could be done with older CMake versions, but that is quite inconvenient.

Since we have a CI it will be trivial to test whether our supported compilers have sufficient C++23 support to build libc++

libcxx/docs/Modules.rst

84–99

This bullet point.

libcxx/modules/CMakeLists.txt.in

49

This is the library for the module std. So next to the .pcm files there is a library component.

libcxx/modules/std/memory.cppm

184–185

I'm not sure, but there are several errors in the modules. I copy-paste the synopis and manually cleaned them, so errors were always bound to occur.

There is a test module_std.sh.cpp which validates the declarations in the include files and in the modules.
This found quite some issues, some parts are not implemented in libc++, the synopsis only sometimes doesn't have all declarations (bitmask types for example), etc/
I'm not sure whether the test flags this one, but I have some patches to fix some of the issue found in libc++ itself.

[aaronmondal]

Yeah we can't use new since it's a keyword.

Actually, what about renaming the new.cppm to __new.cppm? The issue with new.cppm is that you can't grep/ls all files in the libcxx/modules directory and extract the partition names from the filenames that way. Since the partition is named std:__new, naming the file same seems more natural.

The downside of this is that __new doesn't 1:1 correspond to the <new> header name. Since there is no include/__new in the same way that there is e.g. include/__algorithm there is little room for confusion though, and users can't directly import the partitions anyways.

[ldionne]

libcxx/test/std/time/time.syn/formatter_tests.h

22

This doesn't seem necessary at all, it's included above.

[aaronmondal]

Yayyy we have a working hello world https://github.com/eomii/rules_ll/pull/98 🥳

[aaronmondal]

I've just opened https://github.com/llvm/llvm-project/issues/62269 to track progress on the irreproducibility of BMIs. After thinking about this for a while, believe that this could be more of an issue than I initially thought. IMO we should add a warning that Module precompilation currently leaks absolute paths into BMIs.

[ChuanqiXu]

libcxx/docs/Modules.rst

84–99

Oh, this bullet has 3 issues. I thought you referred to one of them.

[ldionne]

As discussed, let's split the cppm files into a separate review so we can get the bulk of the mechanical changes in.

libcxx/modules/std/format.cppm

11

As discussed, I think this makes it too hard to distinguish what is a comment and what is an actual declaration. Thanks so much for doing the experiment -- I think it was a good idea on paper but it turns out to be a bit too hard to read.

libcxx/modules/std/memory.cppm

32

This has been fixed now. Also some stuff in <mutex> and <utility>, see D145589.

[Mordante]

Various cleanups
Documentation updates
Removed magic header from modules, this requires Clang 17

[Mordante]

Rebased.

[Mordante]

Removes some no longer needed work-arounds.
Adds a work-around for as_rvalue, which will be put in a separate review too.

[aaronmondal]

Regarding compilation of the BMIs themselves the only issue left seems to be that <type_traits> doesn't export some symbols and <functional> needs to be included in its cppm at the moment. Other than that everything builds and std is importable.

Some std:: symbols still breaks in a few unsurprising places, e.g. format/ranges.

[Mordante]

In D144994#4325321, @aaronmondal wrote:

Regarding compilation of the BMIs themselves the only issue left seems to be that <type_traits> doesn't export some symbols and <functional> needs to be included in its cppm at the moment. Other than that everything builds and std is importable.

Some std:: symbols still breaks in a few unsurprising places, e.g. format/ranges.

The first part for functional is D149351 which moves these declarations in the proper primary header.
On my list there are some issues with invocable too.

Once these are done I can start to finalize the cppm files.

[Mordante]

Polish the standard modules.

[Mordante]

Rebased on main.

[Mordante]

Final review of the std module

• fixes some minor issues
• enables the declarations that have been recently implemented by various contributors

[Mordante]

I split of the .cppm files for the std module in D151030 once that lands I'll rebase this patch on top of that.

[ldionne]

libcxx/utils/use_modules_in_test.py

2

Let's extract this to its own patch.

160

Could we instead use this annotation in the test itself, with a relevant comment?

// UNSUPPORTED: use_module_std && !(c++11 || c++14 || c++17)

Or actually, we can use just

// UNSUPPORTED: use_module_std

since we never run with use_module_std in C++11/14/17.

170–177

The fact that we need to disable these tests is quite interesting. I think it surfaces a pretty big problem with modules (as a language feature). Today, we use macros and compiler flags to customize the behavior of the library on a per translation-unit basis. That is an extremely important property of some aspects of the library (*). If we want everything to transition to modules (and we do), we need a way to satisfy those use cases. I don't mind whether that's done by defining macros, via some compiler flags or some other way, but those use cases are valid and we need to figure out how we're going to support them. Otherwise, we'll be left with large parts of our user base that can't switch over to C++ modules because they don't support some required use cases.

(*) For example, we use _LIBCPP_ENABLE_ASSERTIONS to control whether assertions are enabled or not on a per-TU basis. This is extremely important: when converting code bases to assertions, users might want to prevent assertions in some performance critical parts of the code, while still enabling assertions everywhere else. This approach allows for an incremental adoption of the feature and that's an important aspect of it being deployable in the real world.

I would like us to discuss avenues we could use to satisfy these use cases. For example, could we ship multiple "versions" of the library, each configured differently, and then the compiler would use a different version based on the compiler flags it is passed? I think pretty much all vendors are going to have similar questions, and that's going to severely limit how useful C++ modules can be in the real world. @jwakely @CaseyCarter I assume that's also something that you folks have thought about?

CC @ChuanqiXu

[ChuanqiXu]

libcxx/utils/use_modules_in_test.py

170–177

(I am not 100% sure that I understand your question)

A background (I guess you already know this. But I'd like to repeat it again to avoid misunderstandings): we're going to ship (*.cppm) files. And the compilation of the std modules would happen in the users' machine. Also the compilation of the std modules may occur multiple times due to different configuration of the build. For example, the Release build and the Debug build will compile std modules twice with different options. And ideally the process would be controlled by the build systems (cmake).

The consensus in SG15 now is that we need to ship source files (*.cppm files) and a metadata file to describe the requirement/information of the library/module. Note that SG15 doesn't have a consensus (or even a feeling as far as I know) how the metadata should be.

So a temporary conclusion here is that possibly there is not an off-the-shelf practice that we can take directly. We would have to explore the method during the practice.

Also, I strongly suggest to send this question to SG15 and there are multiple experts who studied the problem for a relative long time. Maybe it will be more helpful than the small group discussion here.

CC: @ruoso @ben.boeckel

[ldionne]

libcxx/utils/use_modules_in_test.py

170–177

we're going to ship (*.cppm) files. And the compilation of the std modules would happen in the users' machine.

Yes, we're on the same page. I'm not sure why I mentioned shipping various versions of the library, I was thinking about the need to pre-build multiple versions of the library for testing purposes (which is not related). Let me try to clarify my concern, and you can let me know whether I'm getting agitated for no good reason :-). Let's take for example _LIBCPP_ENABLE_ASSERTIONS. Today, that's a macro you define as -D_LIBCPP_ENABLE_ASSERTIONS=1 on the command-line or #define _LIBCPP_ENABLE_ASSERTIONS 1 as the first thing in your file, and the result is that libc++ functions like operator[] will be compiled with some assertions inside. This can be controlled on a per-TU basis today using the #include model we all know.

How do we do that in modules world if you're using import std instead? You can still pass -D_LIBCPP_ENABLE_ASSERTIONS=1 on the command-line, however that won't do anything because the compiler is using the BMI files that have already been compiled and _LIBCPP_ENABLE_ASSERTIONS was set to 0 when those were compiled. Right?

You can also still use #define _LIBCPP_ENABLE_ASSERTIONS=1 as the first thing in your file, but that also won't do anything because import std; doesn't have any way (or desire) to somehow resolve to a different BMI depending on the current macro environment. Right?

Basically, my question is: "What's the WG21-blessed alternative for this use case"? Are we expected to lift this mechanism from -D_LIBCPP_ENABLE_ASSERTIONS=1 to some other compiler flag like -flibcxx-enable-assertions (with a better name)? Then users would tell their build-system to build specific TUs using that flag, and that would result in different BMIs being compiled and used for those TUs? I think that would work, but it would create some challenges for sure.

If that's the expectation, then it means that we either need to build multiple configurations of the library BMIs during CMake invocation for the test suite to reuse, or lit needs to be able to somehow rebuild BMIs (or ask CMake to do it) based on the requirement of various tests in the test suite. For example, you'd tell lit that you're using -fsized-deallocation in your test, so it would go and rebuild the BMIs for libc++ with that set of compiler flags, and then set the appropriate substitution for that test to use the right BMIs for -fsized-deallocation. Possible, but fun to implement :-)

[davidstone]

libcxx/utils/use_modules_in_test.py

170–177

The point that you would define _LIBCPP_ENABLE_ASSERTIONS=1 is when you compile the module in which you want to enable assertions. In other words, you can control it on a per-TU basis for the code that has the assertions, but not for the code using the code that has assertions. In general, you get ODR violations if the user's code is using multiple versions of library code.

So the user would not define the macro in their source file, nor would they (in CMake terms) associate the macro with their own target, they would either define the macro globally in their build system or associate it (somehow) with the standard library build target.

[ChuanqiXu]

libcxx/utils/use_modules_in_test.py

170–177

@ldionne Oh, I got your point. You're mainly talking about testing instead of distributing. (IIUC).

How do we do that in modules world if you're using import std instead? You can still pass -D_LIBCPP_ENABLE_ASSERTIONS=1 on the command-line, however that won't do anything because the compiler is using the BMI files that have already been compiled and _LIBCPP_ENABLE_ASSERTIONS was set to 0 when those were compiled. Right?

You can also still use #define _LIBCPP_ENABLE_ASSERTIONS=1 as the first thing in your file, but that also won't do anything because import std; doesn't have any way (or desire) to somehow resolve to a different BMI depending on the current macro environment. Right?

Yes. This is by design. We don't want the build of modules to be affected by its users.

Basically, my question is: "What's the WG21-blessed alternative for this use case"?

The expected solution may be add a metadata file (which haven't been defined yet) to the library and the metadata file will contain informations about the interested flags of the library.

And if there are two (multiple) users use the same library and all the flags they used in the interesting flag set are the same, they can reuse the same BMI file. But if there is any different flag in the interesting flag set, they have to use different BMI file.

Let's come back to the example of testing of libcxx. Here every lit test with a configuration is a user of the std module. But the same lit test with different configuration are considered to be different users of the std module. And in your example, both -D_LIBCPP_ENABLE_ASSERTIONS and -fsized-deallocation are the interesting flags.

BTW, the intention of the design is that, (I guess you already know this too. Just repeat again to be informative. Sorry if this is wordy), the build system will rebuild the BMIs if the input compilation flag contains -DINTERESTING_MACRO=NEW_VALUE but the build system are capable to reuse the BMIs if the input compilation flag contains -DABCDEFG. This is important because almost every library has local macro definitions.

Given the lit tests are internal users, maybe we can assume all the flags to be interesting flags.

If that's the expectation, then it means that we either need to build multiple configurations of the library BMIs during CMake invocation for the test suite to reuse, or lit needs to be able to somehow rebuild BMIs (or ask CMake to do it) based on the requirement of various tests in the test suite.

If I read correctly, yes.

My mental model for the general case would be:

1. the user (a lit test with a configuration in this example) would ask the build system about the location of the BMI file with a certain module name and a list of flags.
2. The build systems will lookup if the appropriate BMI exists. If yes, return the location. If not, build it and return the location.

And in the testing case, maybe we need to implement a mechanism for lit and cmake to communicate with each other.

[ChuanqiXu]

libcxx/utils/use_modules_in_test.py

170–177

We don't want the build of modules to be affected by its users.

We don't want the build of modules to be affected by its users surprisingly. That said, the build of the modules can be affected by its users if and only if in the way designed by the modules.

[ben.boeckel]

libcxx/utils/use_modules_in_test.py

170–177

We don't want the build of modules to be affected by its users surprisingly.

Alas, consumers do need to agree on a lot of flags. We'll need at least a BMI per consuming standard. Probably per flag like -D_LIBCPP_* and the like, whatever -f and -m flags affect ABI, etc. I'd love for "I build module X one way and everyone agrees", but implementors have told me that it doesn't work that way. The ODR probably pops in at the other extreme as well.

[Mordante]

libcxx/utils/use_modules_in_test.py

170–177

Is there a way to indicate BMIs are incompatible due to ODR violations? For example when the size of a struct changes depending on compiler flags used?

[ben.boeckel]

libcxx/utils/use_modules_in_test.py

170–177

My understanding is that Clang stores the flags used to make the BMI in the BMI and if they mismatch "enough", it is incompatible. Given that…just about every flag does this (e.g., anything that affects __has_attribute, __has_builtin, or other preprocessor state), the BMI compatibility contract is very narrow. Named modules *might* be able to be a bit looser given how preprocessor state is far more isolated, but header units are going to leak these state differences far more easily.

[ChuanqiXu]

libcxx/utils/use_modules_in_test.py

170–177

Currently clang will try to perform a strict check when loading BMIs. For example if a TU imports module A and B, and the same declaration in module A and B differ, clang will emit an error.

For example when the size of a struct changes depending on compiler flags used?

But this may not be possible IIUC.

[DanielaE]

@Mordante As your direction is 'lifting' exported names from the GMF into the module purview I've initiated core issue CWG2783 https://cplusplus.github.io/CWG/issues/2783.html to strengthen the foundations.

[Mordante]

In D144994#4607621, @DanielaE wrote:

@Mordante As your direction is 'lifting' exported names from the GMF into the module purview I've initiated core issue CWG2783 https://cplusplus.github.io/CWG/issues/2783.html to strengthen the foundations.

Thanks @DanielaE! After our last discussion I had looking at this at my todo list.

[Mordante]

Since we move to GitHub PRs we're cleaning up our review queue. I abandon this patch and provide GitHub PRs for the not yet landed parts.

libcxx/CMakeLists.txt

 option(LIBCXX_ENABLE_VENDOR_AVAILABILITY_ANNOTATIONS
   "Whether to turn on vendor availability annotations on declarations that depend
    on definitions in a shared library. By default, we assume that we're not building
    libc++ for any specific vendor, and we disable those annotations. Vendors wishing
    to provide compile-time errors when using features unavailable on some version of
    the shared library they shipped should turn this on and see `include/__availability`
    for more details." OFF)
 option(LIBCXX_ENABLE_CLANG_TIDY "Whether to compile and run clang-tidy checks" OFF)
+option(LIBCXX_ENABLE_STD_MODULE
+   "Whether to enable the building the C++23 std module. This feature is experimental
+    and far from usable. Only enable this when interested in testing and developing
+  this module." OFF)
 
 if (CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
   set(LIBCXX_DEFAULT_TEST_CONFIG "llvm-libc++-shared-gcc.cfg.in")
 elseif(MINGW)
   set(LIBCXX_DEFAULT_TEST_CONFIG "llvm-libc++-mingw.cfg.in")
 elseif(WIN32) # clang-cl
   if (LIBCXX_ENABLE_SHARED)
     set(LIBCXX_DEFAULT_TEST_CONFIG "llvm-libc++-shared-clangcl.cfg.in")
message(FATAL_ERROR "Invalid value for LIBCXX_BENCHMARK_NATIVE_STDLIB: "
             "'${LIBCXX_BENCHMARK_NATIVE_STDLIB}'")
   endif()
 endif()
 
 option(LIBCXX_INCLUDE_DOCS "Build the libc++ documentation." ${LLVM_INCLUDE_DOCS})
 set(LIBCXX_LIBDIR_SUFFIX "${LLVM_LIBDIR_SUFFIX}" CACHE STRING
     "Define suffix of library directory name (32/64)")
 option(LIBCXX_INSTALL_HEADERS "Install the libc++ headers." ON)
+option(LIBCXX_INSTALL_MODULES "Install the libc++ modules." ON)
 option(LIBCXX_INSTALL_LIBRARY "Install the libc++ library." ON)
 cmake_dependent_option(LIBCXX_INSTALL_STATIC_LIBRARY
   "Install the static libc++ library." ON
   "LIBCXX_ENABLE_STATIC;LIBCXX_INSTALL_LIBRARY" OFF)
 cmake_dependent_option(LIBCXX_INSTALL_SHARED_LIBRARY
   "Install the shared libc++ library." ON
   "LIBCXX_ENABLE_SHARED;LIBCXX_INSTALL_LIBRARY" OFF)
 

 
 #===============================================================================
 # Configure System
 #===============================================================================
 
 # TODO: Projects that depend on libc++ should use LIBCXX_GENERATED_INCLUDE_DIR
 # instead of hard-coding include/c++/v1.
 
+# TODO MODULES: CMAKE_INSTALL_MODULEDIR does not exist therefore generate one
+# based on the include dir name.
+string(REPLACE "include" "modules" CMAKE_INSTALL_MODULEDIR ${CMAKE_INSTALL_INCLUDEDIR})
+
 set(LIBCXX_INSTALL_INCLUDE_DIR "${CMAKE_INSTALL_INCLUDEDIR}/c++/v1" CACHE PATH
     "Path where target-agnostic libc++ headers should be installed.")
+set(LIBCXX_INSTALL_MODULE_DIR "${CMAKE_INSTALL_MODULEDIR}/c++/v1" CACHE PATH
+    "Path where target-agnostic libc++ modules should be installed.")
 set(LIBCXX_INSTALL_RUNTIME_DIR "${CMAKE_INSTALL_BINDIR}" CACHE PATH
     "Path where built libc++ runtime libraries should be installed.")
 
 set(LIBCXX_SHARED_OUTPUT_NAME "c++" CACHE STRING "Output name for the shared libc++ runtime library.")
 set(LIBCXX_STATIC_OUTPUT_NAME "c++" CACHE STRING "Output name for the static libc++ runtime library.")
 
 if(LLVM_ENABLE_PER_TARGET_RUNTIME_DIR AND NOT APPLE)
   set(LIBCXX_LIBRARY_DIR ${LLVM_LIBRARY_OUTPUT_INTDIR}/${LLVM_DEFAULT_TARGET_TRIPLE})
   set(LIBCXX_GENERATED_INCLUDE_DIR "${LLVM_BINARY_DIR}/include/c++/v1")
+  set(LIBCXX_GENERATED_MODULE_DIR "${LLVM_BINARY_DIR}/modules/c++/v1")
   set(LIBCXX_GENERATED_INCLUDE_TARGET_DIR "${LLVM_BINARY_DIR}/include/${LLVM_DEFAULT_TARGET_TRIPLE}/c++/v1")
   set(LIBCXX_INSTALL_LIBRARY_DIR lib${LLVM_LIBDIR_SUFFIX}/${LLVM_DEFAULT_TARGET_TRIPLE} CACHE PATH
       "Path where built libc++ libraries should be installed.")
   set(LIBCXX_INSTALL_INCLUDE_TARGET_DIR "${CMAKE_INSTALL_INCLUDEDIR}/${LLVM_DEFAULT_TARGET_TRIPLE}/c++/v1" CACHE PATH
       "Path where target-specific libc++ headers should be installed.")
   if(LIBCXX_LIBDIR_SUBDIR)
     string(APPEND LIBCXX_LIBRARY_DIR /${LIBCXX_LIBDIR_SUBDIR})
     string(APPEND LIBCXX_INSTALL_LIBRARY_DIR /${LIBCXX_LIBDIR_SUBDIR})
   endif()
 else()
   if(LLVM_LIBRARY_OUTPUT_INTDIR)
     set(LIBCXX_LIBRARY_DIR ${LLVM_LIBRARY_OUTPUT_INTDIR})
     set(LIBCXX_GENERATED_INCLUDE_DIR "${LLVM_BINARY_DIR}/include/c++/v1")
+  set(LIBCXX_GENERATED_MODULE_DIR "${LLVM_BINARY_DIR}/modules/c++/v1")
   else()
     set(LIBCXX_LIBRARY_DIR ${CMAKE_BINARY_DIR}/lib${LIBCXX_LIBDIR_SUFFIX})
     set(LIBCXX_GENERATED_INCLUDE_DIR "${CMAKE_BINARY_DIR}/include/c++/v1")
+  set(LIBCXX_GENERATED_MODULE_DIR "${CMAKE_BINARY_DIR}/modules/c++/v1")
   endif()
   set(LIBCXX_GENERATED_INCLUDE_TARGET_DIR "${LIBCXX_GENERATED_INCLUDE_DIR}")
   set(LIBCXX_INSTALL_LIBRARY_DIR lib${LIBCXX_LIBDIR_SUFFIX} CACHE PATH
       "Path where built libc++ libraries should be installed.")
   set(LIBCXX_INSTALL_INCLUDE_TARGET_DIR "${LIBCXX_INSTALL_INCLUDE_DIR}" CACHE PATH
       "Path where target-specific libc++ headers should be installed.")
 endif()
 

 # versions of the headers in order to provide C++03 and legacy ABI definitions.
 # NOTE: The public headers can be used with modules in all other contexts.
 function(cxx_add_module_flags target)
   if (LLVM_ENABLE_MODULES)
     # Ignore that the rest of the modules flags are now unused.
     target_add_compile_flags_if_supported(${target} PUBLIC -Wno-unused-command-line-argument)
     target_compile_options(${target} PUBLIC -fno-modules)
   endif()
+  if(LIBCXX_ENABLE_STD_MODULE)
+    target_add_compile_flags_if_supported(${target} PUBLIC -Wno-reserved-module-identifier)
+  endif()
 endfunction()
 
 # Sanitizer flags =============================================================
 
 function(get_sanitizer_flags OUT_VAR  USE_SANITIZER)
   set(SANITIZER_FLAGS)
   set(USE_SANITIZER "${USE_SANITIZER}")
   # NOTE: LLVM_USE_SANITIZER checks for a UNIX like system instead of MSVC.

 endfunction()
 
 #===============================================================================
 # Setup Source Code And Tests
 #===============================================================================
 add_subdirectory(include)
 add_subdirectory(src)
 add_subdirectory(utils)
+if (LIBCXX_ENABLE_STD_MODULE)
+  add_subdirectory(modules)
+endif()
 
 set(LIBCXX_TEST_DEPS "cxx_experimental")
 
 if (LIBCXX_BUILD_EXTERNAL_THREAD_LIBRARY)
   list(APPEND LIBCXX_TEST_DEPS cxx_external_threads)
 endif()
 
 if (LIBCXX_ENABLE_CLANG_TIDY)
   list(APPEND LIBCXX_TEST_DEPS cxx-tidy)
 endif()
 
+if (LIBCXX_ENABLE_STD_MODULE)
+  list(APPEND LIBCXX_TEST_DEPS generate-cxx-modules)
+endif()
+
 if (LIBCXX_INCLUDE_BENCHMARKS)
   add_subdirectory(benchmarks)
 endif()
 
 if (LIBCXX_INCLUDE_TESTS)
   add_subdirectory(test)
   add_subdirectory(lib/abi)
 endif()
 
 if (LIBCXX_INCLUDE_DOCS)
   add_subdirectory(docs)
 endif()

libcxx/docs/Modules.rst

+.. _ModulesInLibcxx:
+
+=================
+Modules in libc++
+=================
+
+This page contains information regarding C++23 module support in libc++.
+There are two kinds of modules available in Clang
+
+ * `Clang specific modules <https://clang.llvm.org/docs/Modules.html>`_
+ * `C++ modules <https://clang.llvm.org/docs/StandardCPlusPlusModules.html>`_
+
+This page mainly discusses the C++ modules. In C++ there are also header units,
+these are not in this patch.
+
+This information is mainly to describe some of the features of this work in
+progress patch. It is not intended to be committed in this form.
+
+Overview
+========
+
+The module sources are stored in ``.cppm`` files. These need to be available as
+BMIs, which are ``.pcm`` files for Clang. These files are not portable, they
+depend on the compiler used and its compilation flags. Therefore there needs to
+be a way to distribute the ``.cppm`` files to the user and offer a way for them
+to build and use the ``.pcm`` files.
+
+What works
+~~~~~~~~~~
+
+ * Building BMIs
+ * Running tests using modules, not all tests pass yet
+ * Using the ``std`` module in external projects
+ * Using the ``std.compat`` module in external projects
+ * Installing libc++ (the installation location is still being discussed)
+ * The following "parts disabled" configuration options are supported
+
+   * ``LIBCXX_ENABLE_LOCALIZATION``
+   * ``LIBCXX_ENABLE_WIDE_CHARACTERS``
+
+Some of the current limitations
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+
+ * For now everything requires CMake.
+ * The ``.pcm`` files are regenerated when running a ``lit`` invocation.
+ * There is no Buildkite CI job yet. This is work in progress.
+ * The ``std`` should be complete, but there are no tests to verify that.
+ * The ``std.compat`` should be complete, but there are no tests to verify that.
+ * There is an incomplete test that verifies the correctness of modules using
+   the AST matchers. This tests all public declarations in the headers are in
+   the modules.
+ * There is a way to test modules, but it requires changes in the CI run. This
+   is not ideal, but it works.
+ * Requires CMake 3.26
+ * Requires Ninja 1.11
+ * Requires Clang 17 (Clang 16 does not work since some Clang 17 fixes are required)
+ * The path to the compiler may not be a symlink, ``clang-scan-deps`` does
+   not handle that case properly.
+ * Configurations with "parts disabled" that are not explicitly allowed will
+   fail to build a ``.pcm``.
+
+Blockers
+~~~~~~~~
+
+  * libc++
+
+    * There are a lot of failing tests, this needs to be investigated after
+      the global namespace issues have been fixed.
+      Currently 222 of the 7243 test in the directory ``libcxx/test/std`` fail
+      when using the ``std.compat`` module.
+    * CMake 3.26 doesn't work with ``libunwind``. For ``libc++`` and
+      ``libc++abi`` it ignores a lot of compilation flags.
+      `Patch <https://llvm.org/D145596>`__
+    * The ``-pthread`` flag is not used a compile flag. This flag affects
+      the preprocessor so it should be a compile and linker flag.
+      There is a WIP patch.
+    * There are symbols with internal linkage in the header.
+
+      * Due to older language versions don't support ``inline constexpr``.
+        `Patch <https://reviews.llvm.org/D143797>`__
+
+  * Clang
+
+    * ``__synth_three_way_result`` does not work. There is a work-around in libc++.
+
+      * `bug report <https://github.com/llvm/llvm-project/issues/57222>`__
+      * `clang patch <https://reviews.llvm.org/D140002>`__
+      * `libc++ patch </usr/share/c++/modules/v<x>/`__
+
+      The work-around is applied in libc++. So this is no blocker anymore.
+
+    * The ranges Niebloids don't work properly. There is no bug report, this
+      might be related to ``__synth_three_way_result``. Changing
+      ``inline constexpr auto foo = foo_t{}`` to
+      ``inline constexpr foo_t foo{}`` seems to fix it in libc++.
+
+    * Including headers after importing the ``std`` module may fail. This is
+      hard to solve and there is a work-around by first including all headers.
+      `bug report <https://github.com/llvm/llvm-project/issues/61329>`__
+
+    * Specializing templated variables does not work as it should. There are
+      two bug reports
+      `module <https://github.com/llvm/llvm-project/issues/62158>`__
+      `frontend <https://github.com/llvm/llvm-project/issues/62174>`__.
+      The module bug has a work-around for users.
+
+Discussion points
+~~~~~~~~~~~~~~~~~
+
+ * The ``std`` module is a C++23 feature, currently it's available in both
+   ``C++20`` and ``C++23``. The simple reason is that this allows testing with
+   different language versions. (This will be needed once the first ``C++26``
+   features become available.) Does libc++ want to retroactively supply the
+   module in ``C++20`` or restrict it to ``C++23`` and later?
+   Since ``C++20`` was far from complete it has been removed. The testing with
+   different language version is replaced by testing with different parts enabled,
+   this is something libc++ needs to support.
+ * Do we want the feature behind a configuration flag or not? Louis already
+   mentioned he prefers it generally available. Then we need to restrict it on
+   the CMake version used.
+ * I'm not too unhappy with the generic approach for user project. I am quite
+   unhappy with how we need to do tests. I have some ideas how we can improve
+   this, basically by caching multiple configurations. However it would be good
+   to discuss that approach before spending effort in it.
+ * The current approach to make the tests work with modules add a ``#ifdef``
+   to select between including headers or importing the module ``std``.
+   This works great for testing during development, however the libc++
+   maintainers need to decide to keep using this approach in production or
+   that a different solution needs to be used.
+
+
+Design choices
+==============
+
+Shipping
+--------
+
+Since BMIs are not portable libc++ will ship its ``.cppm`` files. Users need to
+compile them to BMIs. Since the feature is under heavy development there is no
+support for build tools. For that reason a CMake file will be shipped. Once
+build tool vendors have support for libc++'s modules this support file will be
+removed. (This file is used for the libc++ tests too, so maybe it remains build
+it will not longer be shipped.)
+
+The modules will shipped in a directory structure similar to the include.
+Includes are stored in ``<PREFIX>/includes/c++/v1`` modules will be stored in
+``<PREFIX>/modules/c++/v1/``.
+
+To make it easy for build tool vendors the directory
+``<PREFIX>/modules/c++/v1/`` will have the following contents
+
+  * ``std.cppm`` the main ``std`` module source.
+  * ``std/`` all other sources for the ``std`` module. Using this directory
+    makes it easy for vendors to glob all files belonging the the ``std``
+    module.
+  * ``std.compat.cppm`` the main ``std.compat`` module source.
+  * ``std.compat/`` like ``std/`` but for the ``std.compat`` module.
+
+If in the future the C++ Standard defines more modules they will get a similar
+treatment. (For example modules for experimental library features.)
+
+.. note:: This matches the current proposal on `discourse
+   <https://discourse.llvm.org/t/rfc-about-the-default-location-for-std-modules/69191/29>`__
+   so it may still change. That discussion starts to move to
+   ``<PREFIX>/share/c++/modules/v1/`` instead of the original location.
+
+Testing
+-------
+
+To properly test all tests with modules all tests need to be converted. The
+script ``libcxx/utils/use_modules_in_test.py`` can do this conversion. Since not
+all developers are happy with the output this script will be executed in the CI
+and then the CI will run on the modified sources.
+
+Running tests
+=============
+
+In order to run the tests with C++23 modules the libc++ needs to be build with
+the following configuration option in CMake ``LIBCXX_ENABLE_STD_MODULE``.
+
+The tests are executed like
+
+.. code-block:: bash
+
+  $ <build>/bin/llvm-lit -sv libcxx/test/std/containers # Run the tests without modules
+  $ <build>/bin/llvm-lit -sv -Denable_modules=clang libcxx/test/std/containers # Run the tests with Clang modules
+  $ <build>/bin/llvm-lit -sv -Denable_modules=std libcxx/test/std/containers # Run the tests with Standard std module
+  $ <build>/bin/llvm-lit -sv -Denable_modules=std.compat libcxx/test/std/containers # Run the tests with Standard std.compat module
+
+The Clang modules are the modules that have been available for many libc++ releases.
+The Standard modules are the new feature.
+
+Using in external projects
+==========================
+
+Users need to be able to build their own BMI files.
+
+.. note:: The requirements for users to build their own BMI files will remain
+   true for the forseeable future. For now this needs to be done manually.
+   Once libc++'s implementation is more mature we will reach out to build
+   system vendors, with the goal that the building of the BMI files will be
+   done by the build system.
+
+Currently this requires a local build of libc++ with modules enabled.
+"Importing" libc++ is a project can be done with the following CMake code.
+
+.. code-block:: cmake
+
+  include(FetchContent)
+  FetchContent_Declare(
+    std
+    URL file://<build>/include/c++/modules/
+    DOWNLOAD_EXTRACT_TIMESTAMP TRUE
+  )
+  FetchContent_GetProperties(std)
+  if(NOT std_POPULATED)
+    # The C++ version used in your project must match the version used here.
+    set(CMAKE_CXX_STANDARD 23)
+    # These compiler flags must match the flags used in your project.
+    # When they don't match the generated BMIs can't be used in your project.
+    set(CMAKE_CXX_FLAGS "-fexperimental-library")
+    FetchContent_Populate(std)
+    add_subdirectory(${std_SOURCE_DIR} ${std_BINARY_DIR} EXCLUDE_FROM_ALL)
+  endif()
+
+The libraries and/or executables that use the ``std`` module need to link
+against the ``std`` library. This is needed for CMake to get the proper module
+dependencies. Similar for the ``std.compat`` module.
+
+.. warning:: There is a bug in Clang where including headers after an import
+   may fail. Instead include all headers first and then include the module.
+
+
+.. code-block:: cmake
+
+   #include <vector>
+
+   #include "A header that may include other headers.h"
+
+   import std;
+
+   ...

libcxx/modules/CMakeLists.txt.in

+# CMake 3.25 has modules support, but that does not work properly.
+cmake_minimum_required(VERSION 3.26)
+
+project(libc++-std LANGUAGES CXX)
+
+# Enable CMake's module support
+set(CMAKE_EXPERIMENTAL_CXX_MODULE_CMAKE_API "2182bf5c-ef0d-489a-91da-49dbc3090d2a")
+set(CMAKE_EXPERIMENTAL_CXX_MODULE_DYNDEP 1)
+
+# Default to C++ extensions being off. Clang's modules support have trouble
+# with extensions right now.
+set(CMAKE_CXX_EXTENSIONS OFF)
+
+# Propagates the CMake options to the modules.
+#
+# This uses the std module hard-coded since the std.compat module does not
+# depend on these flags.
+macro(compile_define_if_not condition def)
+  if (NOT ${condition})
+    target_compile_definitions(std PRIVATE ${def})
+  endif()
+endmacro()
+macro(compile_define_if condition def)
+  if (${condition})
+    target_compile_definitions(std PRIVATE ${def})
+  endif()
+endmacro()
+
+if(NOT @LIBCXX_ENABLE_THREADS@ OR NOT @LIBCXXABI_ENABLE_THREADS@ OR NOT @LIBCXX_ENABLE_MONOTONIC_CLOCK@)
+  message(FATAL_ERROR "Modules without thread support is not yet implemented.")
+endif()
+if(NOT @LIBCXX_ENABLE_FILESYSTEM@)
+  message(FATAL_ERROR "Modules without filesystem support is not yet implemented.")
+endif()
+if(NOT @LIBCXX_ENABLE_RANDOM_DEVICE@)
+  message(FATAL_ERROR "Modules without randome device support is not yet implemented.")
+endif()
+if(NOT @LIBCXX_ENABLE_FSTREAM@)
+  message(FATAL_ERROR "Modules without fstream support is not yet implemented.")
+endif()
+if(NOT @LIBCXX_ENABLE_UNICODE@)
+  message(FATAL_ERROR "Modules without Unicode support is not yet implemented.")
+endif()
+if(NOT @LIBCXX_ENABLE_EXCEPTIONS@ OR NOT @LIBCXXABI_ENABLE_EXCEPTIONS@)
+  message(FATAL_ERROR "Modules without exception support is not yet implemented.")
+endif()
+
+add_library(std)
+target_sources(std
+  PUBLIC FILE_SET cxx_modules TYPE CXX_MODULES FILES
+    @LIBCXX_SOURCES_MODULE_STD@
+)
+
+target_compile_definitions(std PRIVATE _LIBCPP_ENABLE_EXPERIMENTAL)
+target_include_directories(std SYSTEM PRIVATE @LIBCXX_CONFIGURED_INCLUDE_DIR@)
+
+target_compile_options(std
+  PUBLIC
+    -nostdinc++
+    -Wno-reserved-module-identifier
+    @LIBCXX_COMPILE_FLAGS@
+)
+
+add_library(std.compat)
+target_sources(std.compat
+  PUBLIC FILE_SET cxx_modules TYPE CXX_MODULES FILES
+    @LIBCXX_SOURCES_MODULE_STD_COMPAT@
+)
+target_include_directories(std.compat SYSTEM PRIVATE @LIBCXX_CONFIGURED_INCLUDE_DIR@)
+target_compile_options(std.compat
+  PUBLIC
+    -nostdinc++
+    -Wno-reserved-module-identifier
+    # HACK to find the std module
+    -fmodule-file=std=CMakeFiles/std.dir/std.pcm
+    @LIBCXX_COMPILE_FLAGS@
+)
+target_link_libraries(std.compat PRIVATE std)

libcxx/modules/std.compat/cassert.cppm

+// -*- C++ -*-
 //===----------------------------------------------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
 
-// <utility>
+module;
+#include <cassert>
 
-// template<class T, T N>
-//   using make_integer_sequence = integer_sequence<T, 0, 1, ..., N-1>;
-
-// UNSUPPORTED: c++03, c++11
-
-#define _LIBCPP_TESTING_FALLBACK_MAKE_INTEGER_SEQUENCE
-#include "make_integer_seq.pass.cpp"
-
-#include "test_macros.h"
+export module std.compat:cassert;
+export {
+  // This module exports nothing.
+} // export

libcxx/modules/std/format.cppm

+// -*- C++ -*-
+//===----------------------------------------------------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+
+module;
+#include <format>
+
+export module std:format;
+#if !defined(_LIBCPP_HAS_NO_INCOMPLETE_FORMAT)
+export namespace std {
+
+  // [format.context], class template basic_format_context
+  using std::basic_format_context;
+  using std::format_context;
+#  ifndef _LIBCPP_HAS_NO_WIDE_CHARACTERS
+  using std::wformat_context;
+#  endif
+
+  // [format.args], class template basic_format_args
+  using std::basic_format_args;
+  using std::format_args;
+#  ifndef _LIBCPP_HAS_NO_WIDE_CHARACTERS
+  using std::wformat_args;
+#  endif
+
+  // [format.fmt.string], class template basic_format_string
+  using std::basic_format_string;
+  using std::format_string;
+#  ifndef _LIBCPP_HAS_NO_WIDE_CHARACTERS
+  using std::wformat_string;
+#  endif
+
+  // [format.functions], formatting functions
+  using std::format;
+  using std::format_to;
+  using std::vformat;
+  using std::vformat_to;
+
+  using std::format_to_n;
+  using std::format_to_n_result;
+  using std::formatted_size;
+
+  // [format.formatter], formatter
+  using std::formatter;
+
+  // [format.formattable], concept formattable
+  using std::formattable;
+
+  // [format.parse.ctx], class template basic_format_parse_context
+  using std::basic_format_parse_context;
+  using std::format_parse_context;
+#  ifndef _LIBCPP_HAS_NO_WIDE_CHARACTERS
+  using std::wformat_parse_context;
+#  endif
+
+  // [format.range], formatting of ranges
+  // [format.range.fmtkind], variable template format_kind
+  using std::format_kind;
+  using std::range_format;
+
+  // [format.range.formatter], class template range_formatter
+  using std::range_formatter;
+
+  // [format.arg], class template basic_format_arg
+  using std::basic_format_arg;
+  using std::visit_format_arg;
+
+  // [format.arg.store], class template format-arg-store
+  using std::make_format_args;
+#  ifndef _LIBCPP_HAS_NO_WIDE_CHARACTERS
+  using std::make_wformat_args;
+#  endif
+
+  // [format.error], class format_error
+  using std::format_error;
+} // namespace std
+#endif // !defined(_LIBCPP_HAS_NO_INCOMPLETE_FORMAT)

libcxx/modules/std/memory.cppm

+// -*- C++ -*-
+//===----------------------------------------------------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+
+module;
+#include <memory>
+
+export module std:memory;
+export namespace std {
+  // [pointer.traits], pointer traits
+  using std::pointer_traits;
+
+  // [pointer.conversion], pointer conversion
+  using std::to_address;
+
+  // [ptr.align], pointer alignment
+  using std::align;
+  using std::assume_aligned;
+
+  // [obj.lifetime], explicit lifetime management
+  //  using std::start_lifetime_as; NOT IMPLEMENTED
+  //  using std::start_lifetime_as_array NOT IMPLEMENTED;
+
+  // [allocator.tag], allocator argument tag
+  //  using std::allocator_arg; TODO INTERNAL LINKAGE
+  //  using std::allocator_arg_t TODO INTERNAL LINKAGE;
+
+  // [allocator.uses], uses_allocator
+  using std::uses_allocator;
+
+  // [allocator.uses.trait], uses_allocator
+  using std::uses_allocator_v;
+
+  // [allocator.uses.construction], uses-allocator construction
+  using std::uses_allocator_construction_args;
+
+  using std::make_obj_using_allocator;
+  using std::uninitialized_construct_using_allocator;
+
+  // [allocator.traits], allocator traits
+  using std::allocator_traits; // freestanding
+
+  using std::allocation_result;
+
+  using std::allocate_at_least;
+
+  // [default.allocator], the default allocator
+  using std::allocator;
+  using std::operator==;
+
+  // [specialized.addressof], addressof
+  using std::addressof;
+
+  // [specialized.algorithms], specialized algorithms
+  // [special.mem.concepts], special memory concepts
+
+  using std::uninitialized_default_construct;
+  using std::uninitialized_default_construct_n;
+
+  namespace ranges {
+    using std::ranges::uninitialized_default_construct;
+    using std::ranges::uninitialized_default_construct_n;
+  } // namespace ranges
+
+  using std::uninitialized_value_construct;
+  using std::uninitialized_value_construct_n;
+
+  namespace ranges {
+    using std::ranges::uninitialized_value_construct;
+    using std::ranges::uninitialized_value_construct_n;
+  } // namespace ranges
+
+  using std::uninitialized_copy;
+  using std::uninitialized_copy_n;
+
+  namespace ranges {
+    using std::ranges::uninitialized_copy;
+    using std::ranges::uninitialized_copy_result;
+
+    using std::ranges::uninitialized_copy_n;
+    using std::ranges::uninitialized_copy_n_result;
+
+  } // namespace ranges
+  using std::uninitialized_move;
+  using std::uninitialized_move_n;
+
+  namespace ranges {
+    using std::ranges::uninitialized_move;
+    using std::ranges::uninitialized_move_result;
+
+    using std::ranges::uninitialized_move_n;
+    using std::ranges::uninitialized_move_n_result;
+
+  } // namespace ranges
+
+  using std::uninitialized_fill;
+  using std::uninitialized_fill_n;
+
+  namespace ranges {
+    using std::ranges::uninitialized_fill;
+    using std::ranges::uninitialized_fill_n;
+  } // namespace ranges
+
+  // [specialized.construct], construct_at
+  using std::construct_at;
+
+  namespace ranges {
+    using std::ranges::construct_at;
+  }
+  // [specialized.destroy], destroy
+  using std::destroy;
+  using std::destroy_at;
+  using std::destroy_n;
+
+  namespace ranges {
+    using std::ranges::destroy;
+    using std::ranges::destroy_at;
+    using std::ranges::destroy_n;
+  } // namespace ranges
+    // [unique.ptr], class template unique_ptr
+  using std::default_delete;
+  using std::unique_ptr;
+
+  using std::make_unique;
+  using std::make_unique_for_overwrite;
+
+  using std::swap;
+
+  using std::operator!=;
+  using std::operator<;
+  using std::operator>;
+  using std::operator<=;
+  using std::operator>=;
+  using std::operator<=>;
+
+  using std::operator<<;
+
+  // [util.smartptr.weak.bad], class bad_weak_ptr
+  using std::bad_weak_ptr;
+
+  // [util.smartptr.shared], class template shared_ptr
+  using std::shared_ptr;
+
+  // [util.smartptr.shared.create], shared_ptr creation
+  using std::allocate_shared;
+  using std::allocate_shared_for_overwrite;
+  using std::make_shared;
+  using std::make_shared_for_overwrite;
+
+  // [util.smartptr.shared.spec], shared_ptr specialized algorithms
+  using std::swap;
+
+  // [util.smartptr.shared.cast], shared_ptr casts
+  using std::const_pointer_cast;
+  using std::dynamic_pointer_cast;
+  using std::reinterpret_pointer_cast;
+  using std::static_pointer_cast;
+
+  using std::get_deleter;
+
+  // [util.smartptr.shared.io], shared_ptr I/O
+
+  // [util.smartptr.weak], class template weak_ptr
+  using std::weak_ptr;
+
+  // [util.smartptr.weak.spec], weak_ptr specialized algorithms
+
+  // [util.smartptr.ownerless], class template owner_less
+  using std::owner_less;
+
+  // [util.smartptr.enab], class template enable_shared_from_this
+  using std::enable_shared_from_this;
+
+  // [util.smartptr.hash], hash support
+  using std::hash;
+
+  // [util.smartptr.atomic], atomic smart pointers
+  // Not implemented yet
+  // using std::atomic;
+
+  // [out.ptr.t], class template out_ptr_t
+  //  using std::out_ptr_t NOT IMPLEMENTED;
+
+  // [out.ptr], function template out_ptr
+  //  using std::out_ptr NOT IMPLEMENTED;
+
+  // [inout.ptr.t], class template inout_ptr_t
+  //  using std::inout_ptr_t NOT IMPLEMENTED;
+
+  // [inout.ptr], function template inout_ptr
+  //  using std::inout_ptr NOT IMPLEMENTED;
+
+  // [depr.util.smartptr.shared.atomic]
+  using std::atomic_is_lock_free;
+
+  using std::atomic_load;
+  using std::atomic_load_explicit;
+
+  using std::atomic_store;
+  using std::atomic_store_explicit;
+
+  using std::atomic_exchange;
+  using std::atomic_exchange_explicit;
+
+  using std::atomic_compare_exchange_strong;
+  using std::atomic_compare_exchange_strong_explicit;
+  using std::atomic_compare_exchange_weak;
+  using std::atomic_compare_exchange_weak_explicit;
+
+} // namespace std

libcxx/test/libcxx/language.support/support.types/cstddef.compile.pass.cpp

 //===----------------------------------------------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
 
 // Make sure libc++'s <cstddef> defines types like ::nullptr_t in the global namespace.
 // This is a conforming extension to be consistent with other implementations, which all
 // appear to provide that behavior too.
 
-#include <cstddef>
+// XFAIL: use_module_std
+
+#if defined(TEST_USE_MODULE_STD)
+import std;
+#elif defined(TEST_USE_MODULE_STD_COMPAT)
+import std.compat;
+#else
+#  include <cstddef>
+#endif
 #include "test_macros.h"
 
 using PtrdiffT = ::ptrdiff_t;
 using SizeT = ::size_t;
 #if TEST_STD_VER >= 11
 using MaxAlignT = ::max_align_t;
 #endif
 
+#ifndef TEST_USE_MODULE
 // Supported in C++03 mode too for backwards compatibility with previous versions of libc++
 using NullptrT = ::nullptr_t;
+#endif
 
 // Also ensure that we provide std::nullptr_t in C++03 mode, which is an extension too.
 using StdNullptrT = std::nullptr_t;

libcxx/test/libcxx/module_std.sh.cpp

+//===----------------------------------------------------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+
+// UNSUPPORTED: c++03, c++11, c++14, c++17, c++20
+
+// REQUIRES: has-clang-tidy
+// REQUIRES: use_module_std
+
+// The GCC compiler flags are not always compatible with clang-tidy.
+// UNSUPPORTED: gcc
+
+// RUN %{clang-tidy} %s  --checks='-*,libcpp-header-exportable-declarations' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags}
+// should the headers list just be a list of regexes where symbols from are available...
+
+/*
+BEGIN-SCRIPT
+
+# Used because the sequence of tokens RUN : can't appear anywhere or it'll confuse Lit.
+RUN = "RUN"
+
+### Remove the headers that have no module associated with them
+
+# Note all C-headers using .h are filtered in the loop.
+
+# These headers are not available in C++23, but in older language Standards.
+toplevel_headers.remove('ccomplex')
+toplevel_headers.remove('ciso646')
+toplevel_headers.remove('cstdbool')
+toplevel_headers.remove('ctgmath')
+
+# This header only has forward declarations their declarations are imported in other modules.
+toplevel_headers.remove('iosfwd')
+
+
+# Ignore several declarations found in the includes.
+#
+# Part of these items are bugs other are not yet implemented features.
+SkipDeclarations = dict()
+# See comment in the header.
+SkipDeclarations['cuchar'] = "std::mbstate_t std::size_t"
+# TODO use inline constexpr
+SkipDeclarations['functional'] = "std::placeholders::_1 std::placeholders::_2 std::placeholders::_3 std::placeholders::_4 std::placeholders::_5 std::placeholders::_6 std::placeholders::_7 std::placeholders::_8 std::placeholders::_9 std::placeholders::_10"
+# The operators are added for private types like __iom_t10.
+SkipDeclarations['iomanip'] = "std::operator<< std::operator>>"
+
+
+# This is a specialization for a private type
+SkipDeclarations['iterator'] = "std::pointer_traits"
+
+# TODO
+# This definition is declared in string and defined in isteam
+# This declaration should be part of string
+SkipDeclarations['istream'] = "std::getline"
+
+# P1614 (at many places) and LWG3519 too.
+SkipDeclarations['random'] = "std::operator!="
+# LWG3519 makes these hidden friends. Note the older versions had the requirement of these operations but not in the synopsis.
+SkipDeclarations['random'] = SkipDeclarations['random'] + " std::operator<< std::operator>> std::operator=="
+
+# Declared in the forward header since std::string uses std::allocator
+SkipDeclarations['string'] = "std::allocator"
+# TODO This type should be private
+SkipDeclarations['valarray'] = "std::sliceExpr"
+# TODO use inline constexpr
+SkipDeclarations['utility'] = "std::piecewise_construct"
+# TODO remove zombie names
+# https://libcxx.llvm.org/Status/Cxx20.html#note-p0619
+SkipDeclarations['memory'] = "std::return_temporary_buffer std::get_temporary_buffer"
+# TODO use inline constexpr
+SkipDeclarations['memory'] = SkipDeclarations['memory'] + " std::allocator_arg std::allocator_arg_t"
+# TODO use inline constexpr
+SkipDeclarations['mutex'] = "std::adopt_lock std::defer_lock std::try_to_lock"
+# Internal linkage
+SkipDeclarations['ranges'] = "std::ranges::as_rvalue_view std::ranges::views::as_rvalue"
+
+# TODO this should be part of ios instead
+SkipDeclarations['streambuf'] = "std::basic_ios"
+
+# include/__type_traits/is_swappable.h
+SkipDeclarations['type_traits'] = "std::swap"
+# TODO gotten through __functional/unwrap_ref.h
+SkipDeclarations['type_traits'] =  SkipDeclarations['type_traits'] + " std::reference_wrapper"
+
+# Add delarations in headers.
+# Some headers have their defines in a different header, which may have additional declarations
+ExtraDeclarations = dict()
+# This declaration is in the ostream header.
+ExtraDeclarations['system_error'] = "std::operator<<"
+
+
+# Adds an extra header file to scan
+ExtraHeader = dict()
+# locale has a file and not a subdirectory
+ExtraHeader['locale'] = "v1/__locale$"
+ExtraHeader['thread'] = "v1/__threading_support$"
+
+
+### Remove headers with support issues
+
+# Note these headers should be validated for their modules once the issues have been resolved.
+
+# Headers under evaluation
+toplevel_headers.remove('condition_variable')
+toplevel_headers.remove('mutex')
+
+toplevel_headers.remove('cstdint') # TODO this is a missing include
+toplevel_headers.remove('cuchar') # TODO review this header
+toplevel_headers.remove('cwchar') # TODO review this header, it has comments
+
+# we could move these fwd parts to __fwd/fstream.h and include these from iosfwd which would be consistent with the other newer forward parts
+toplevel_headers.remove('ios')
+toplevel_headers.remove('fstream')
+toplevel_headers.remove('istream')
+toplevel_headers.remove('ostream')
+toplevel_headers.remove('iostream')
+toplevel_headers.remove('sstream')
+toplevel_headers.remove('streambuf')
+
+# These headers contain operator!= that need to be evaluated
+# P1614 probably should have removed them, however that paper is partly implemented.
+toplevel_headers.remove('memory_resource')
+toplevel_headers.remove('complex')
+toplevel_headers.remove('unordered_set')
+toplevel_headers.remove('unordered_map')
+
+# These headers use the spaceship of P1614.
+# Some progress has been made upstream in this area, before removing it from the export
+# list first revase this patch.
+toplevel_headers.remove('array')
+toplevel_headers.remove('deque')
+toplevel_headers.remove('forward_list')
+toplevel_headers.remove('map')
+toplevel_headers.remove('optional')
+toplevel_headers.remove('queue')
+toplevel_headers.remove('regex')
+toplevel_headers.remove('scoped_allocator')
+toplevel_headers.remove('set')
+toplevel_headers.remove('stack')
+toplevel_headers.remove('thread')
+toplevel_headers.remove('vector')
+
+# The header provides the following entries not in the module
+# using std::operator!=;
+# using std::operator<<;
+# using std::operator==;
+# using std::operator>>;
+#
+# The first is probably P1614
+# The other 3 are hidden friend in the Standard but not in the header's synopis.
+# Maybe a paper not implemented?
+toplevel_headers.remove('random')
+
+# This header also provides declarations in the namespace that might be an error
+# using std::operator!=;
+# using std::operator==;
+toplevel_headers.remove('filesystem')
+
+# This header needs review, but first the placeholders (_1, etc) need to be available
+toplevel_headers.remove('functional')
+
+
+# This exports friend like
+#    template <class _CharT, class _Traits, class _Alloc> friend class basic_string;
+#    template <class _Tp, class _Alloc> friend class _LIBCPP_TEMPLATE_VIS vector;
+#    template <class _Tp, size_t> friend class _LIBCPP_TEMPLATE_VIS span;
+# The script needs to be adapted not to do that,
+# Note the other issues need to be investigated
+toplevel_headers.remove('iterator')
+
+# the C++11 isblank is missing, this seems like a bug since there is no test either.
+# libcxx/test/std/localization/locales/locale.convenience/classification
+toplevel_headers.remove('locale')
+
+# The header contains the following zombie names
+# return_temporary_buffer
+# get_temporary_buffer
+# did we forget to implement some papers?
+# Some parts have internal linker issues
+toplevel_headers.remove('memory')
+
+# Some parts have been refactored and might be fixed upstream
+toplevel_headers.remove('system_error')
+
+# a part is in __fwd/foo.h
+toplevel_headers.remove('ranges')
+
+# also has some oddities
+toplevel_headers.remove('tuple')
+
+# needs review
+toplevel_headers.remove('type_traits')
+toplevel_headers.remove('utility')
+
+# Create empty file with all parts.
+print(f"// {RUN}: echo -n > %t.parts")
+
+# Validate all module parts.
+for i, header in enumerate(toplevel_headers):
+  if header.endswith('.h'): # Skip C compatibility headers
+    continue
+
+  # Dump the information as found in the module's cppm file.
+  print(f"// {RUN}: %{{clang-tidy}} %{{module}}/std/{header}.cppm --checks='-*,libcpp-header-exportable-declarations' "
+        f"-config='{{CheckOptions: [ {{key: libcpp-header-exportable-declarations.Header, value: {header}.cppm}} ]}}' "
+        f"--load=%{{test-tools}}/clang_tidy_checks/libcxx-tidy.plugin -- %{{compile_flags}} "
+        f"| sort > %t.{header}.module")
+  print(f"// {RUN}: cat  %t.{header}.module >> %t.parts")
+
+  # Dump the information as found in the module by using the header file(s).
+  skip_declarations = SkipDeclarations.get(header, "")
+  if len(skip_declarations):
+      skip_declarations = f", {{key: libcpp-header-exportable-declarations.SkipDeclarations, value: \"{skip_declarations}\"}}"
+
+  extra_declarations = ExtraDeclarations.get(header, "")
+  if len(extra_declarations):
+      extra_declarations = f", {{key: libcpp-header-exportable-declarations.ExtraDeclarations, value: \"{extra_declarations}\"}}"
+
+  extra_header = ExtraHeader.get(header, "")
+  if len(extra_header):
+      extra_header = f", {{key: libcpp-header-exportable-declarations.ExtraHeader, value: \"{extra_header}\"}}"
+
+  print(f"// {RUN}: %{{clang-tidy}} %s --checks='-*,libcpp-header-exportable-declarations' "
+        "-config='{CheckOptions: [ "
+    f"{{key: libcpp-header-exportable-declarations.Header, value: {header}}} "
+        ", {key: libcpp-header-exportable-declarations.StaticAssertFile, value: \"%t.static_assert.cpp\"} "
+    f"{skip_declarations} {extra_declarations} {extra_header} "
+    "]}' "
+        f"--load=%{{test-tools}}/clang_tidy_checks/libcxx-tidy.plugin -- %{{compile_flags}} -DTEST_{i} "
+        f"| sort > %t.{header}.include")
+
+  # Compare the cppm and header file(s) return the same results.
+  print(f"// {RUN}: diff -u %t.{header}.module %t.{header}.include")
+
+  # The actual include script.
+  print(f"#if defined(TEST_{i})")
+  print(f"#include <{header}>")
+  print("#endif")
+
+# Validate the module parts against the main module.
+
+# Merge the data of the parts
+print(f"// {RUN}: sort -u -o %t.parts %t.parts")
+
+# Dump the information as found in std.cppm.
+#
+# XXX This would require clang-tidy 17 since the the PCH files are not compatible with different versions
+###print(f"// {RUN}: %{{clang-tidy}} %{{module}}/std.cppm --checks='-*,libcpp-header-exportable-declarations' "
+###      f"-config='{{CheckOptions: [ {{key: libcpp-header-exportable-declarations.Header, value: std.cppm}} ]}}' "
+###      f"--load=%{{test-tools}}/clang_tidy_checks/libcxx-tidy.plugin -- %{{compile_flags}} "
+###      f"| sort > %t.module")
+
+# Compare the sum of the parts with the main module.
+###print(f"// {RUN}: diff -u %t.parts %t.module")
+
+END-SCRIPT
+*/
+
+// DO NOT MANUALLY EDIT ANYTHING BETWEEN THE MARKERS BELOW
+// GENERATED-MARKER
+// RUN: echo -n > %t.parts
+// RUN: %{clang-tidy} %{module}/std/algorithm.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: algorithm.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.algorithm.module
+// RUN: cat  %t.algorithm.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: algorithm}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_0 | sort > %t.algorithm.include
+// RUN: diff -u %t.algorithm.module %t.algorithm.include
+#if defined(TEST_0)
+#include <algorithm>
+#endif
+// RUN: %{clang-tidy} %{module}/std/any.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: any.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.any.module
+// RUN: cat  %t.any.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: any}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_1 | sort > %t.any.include
+// RUN: diff -u %t.any.module %t.any.include
+#if defined(TEST_1)
+#include <any>
+#endif
+// RUN: %{clang-tidy} %{module}/std/atomic.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: atomic.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.atomic.module
+// RUN: cat  %t.atomic.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: atomic}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_2 | sort > %t.atomic.include
+// RUN: diff -u %t.atomic.module %t.atomic.include
+#if defined(TEST_2)
+#include <atomic>
+#endif
+// RUN: %{clang-tidy} %{module}/std/barrier.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: barrier.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.barrier.module
+// RUN: cat  %t.barrier.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: barrier}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_3 | sort > %t.barrier.include
+// RUN: diff -u %t.barrier.module %t.barrier.include
+#if defined(TEST_3)
+#include <barrier>
+#endif
+// RUN: %{clang-tidy} %{module}/std/bit.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: bit.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.bit.module
+// RUN: cat  %t.bit.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: bit}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_4 | sort > %t.bit.include
+// RUN: diff -u %t.bit.module %t.bit.include
+#if defined(TEST_4)
+#include <bit>
+#endif
+// RUN: %{clang-tidy} %{module}/std/bitset.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: bitset.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.bitset.module
+// RUN: cat  %t.bitset.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: bitset}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_5 | sort > %t.bitset.include
+// RUN: diff -u %t.bitset.module %t.bitset.include
+#if defined(TEST_5)
+#include <bitset>
+#endif
+// RUN: %{clang-tidy} %{module}/std/cassert.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cassert.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.cassert.module
+// RUN: cat  %t.cassert.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cassert}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_6 | sort > %t.cassert.include
+// RUN: diff -u %t.cassert.module %t.cassert.include
+#if defined(TEST_6)
+#include <cassert>
+#endif
+// RUN: %{clang-tidy} %{module}/std/cctype.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cctype.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.cctype.module
+// RUN: cat  %t.cctype.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cctype}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_7 | sort > %t.cctype.include
+// RUN: diff -u %t.cctype.module %t.cctype.include
+#if defined(TEST_7)
+#include <cctype>
+#endif
+// RUN: %{clang-tidy} %{module}/std/cerrno.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cerrno.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.cerrno.module
+// RUN: cat  %t.cerrno.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cerrno}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_8 | sort > %t.cerrno.include
+// RUN: diff -u %t.cerrno.module %t.cerrno.include
+#if defined(TEST_8)
+#include <cerrno>
+#endif
+// RUN: %{clang-tidy} %{module}/std/cfenv.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cfenv.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.cfenv.module
+// RUN: cat  %t.cfenv.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cfenv}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_9 | sort > %t.cfenv.include
+// RUN: diff -u %t.cfenv.module %t.cfenv.include
+#if defined(TEST_9)
+#include <cfenv>
+#endif
+// RUN: %{clang-tidy} %{module}/std/cfloat.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cfloat.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.cfloat.module
+// RUN: cat  %t.cfloat.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cfloat}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_10 | sort > %t.cfloat.include
+// RUN: diff -u %t.cfloat.module %t.cfloat.include
+#if defined(TEST_10)
+#include <cfloat>
+#endif
+// RUN: %{clang-tidy} %{module}/std/charconv.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: charconv.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.charconv.module
+// RUN: cat  %t.charconv.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: charconv}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_11 | sort > %t.charconv.include
+// RUN: diff -u %t.charconv.module %t.charconv.include
+#if defined(TEST_11)
+#include <charconv>
+#endif
+// RUN: %{clang-tidy} %{module}/std/chrono.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: chrono.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.chrono.module
+// RUN: cat  %t.chrono.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: chrono}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_12 | sort > %t.chrono.include
+// RUN: diff -u %t.chrono.module %t.chrono.include
+#if defined(TEST_12)
+#include <chrono>
+#endif
+// RUN: %{clang-tidy} %{module}/std/cinttypes.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cinttypes.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.cinttypes.module
+// RUN: cat  %t.cinttypes.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cinttypes}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_13 | sort > %t.cinttypes.include
+// RUN: diff -u %t.cinttypes.module %t.cinttypes.include
+#if defined(TEST_13)
+#include <cinttypes>
+#endif
+// RUN: %{clang-tidy} %{module}/std/climits.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: climits.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.climits.module
+// RUN: cat  %t.climits.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: climits}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_14 | sort > %t.climits.include
+// RUN: diff -u %t.climits.module %t.climits.include
+#if defined(TEST_14)
+#include <climits>
+#endif
+// RUN: %{clang-tidy} %{module}/std/clocale.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: clocale.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.clocale.module
+// RUN: cat  %t.clocale.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: clocale}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_15 | sort > %t.clocale.include
+// RUN: diff -u %t.clocale.module %t.clocale.include
+#if defined(TEST_15)
+#include <clocale>
+#endif
+// RUN: %{clang-tidy} %{module}/std/cmath.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cmath.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.cmath.module
+// RUN: cat  %t.cmath.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cmath}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_16 | sort > %t.cmath.include
+// RUN: diff -u %t.cmath.module %t.cmath.include
+#if defined(TEST_16)
+#include <cmath>
+#endif
+// RUN: %{clang-tidy} %{module}/std/codecvt.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: codecvt.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.codecvt.module
+// RUN: cat  %t.codecvt.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: codecvt}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_17 | sort > %t.codecvt.include
+// RUN: diff -u %t.codecvt.module %t.codecvt.include
+#if defined(TEST_17)
+#include <codecvt>
+#endif
+// RUN: %{clang-tidy} %{module}/std/compare.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: compare.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.compare.module
+// RUN: cat  %t.compare.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: compare}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_18 | sort > %t.compare.include
+// RUN: diff -u %t.compare.module %t.compare.include
+#if defined(TEST_18)
+#include <compare>
+#endif
+// RUN: %{clang-tidy} %{module}/std/concepts.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: concepts.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.concepts.module
+// RUN: cat  %t.concepts.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: concepts}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_20 | sort > %t.concepts.include
+// RUN: diff -u %t.concepts.module %t.concepts.include
+#if defined(TEST_20)
+#include <concepts>
+#endif
+// RUN: %{clang-tidy} %{module}/std/coroutine.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: coroutine.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.coroutine.module
+// RUN: cat  %t.coroutine.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: coroutine}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_21 | sort > %t.coroutine.include
+// RUN: diff -u %t.coroutine.module %t.coroutine.include
+#if defined(TEST_21)
+#include <coroutine>
+#endif
+// RUN: %{clang-tidy} %{module}/std/csetjmp.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: csetjmp.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.csetjmp.module
+// RUN: cat  %t.csetjmp.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: csetjmp}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_22 | sort > %t.csetjmp.include
+// RUN: diff -u %t.csetjmp.module %t.csetjmp.include
+#if defined(TEST_22)
+#include <csetjmp>
+#endif
+// RUN: %{clang-tidy} %{module}/std/csignal.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: csignal.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.csignal.module
+// RUN: cat  %t.csignal.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: csignal}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_23 | sort > %t.csignal.include
+// RUN: diff -u %t.csignal.module %t.csignal.include
+#if defined(TEST_23)
+#include <csignal>
+#endif
+// RUN: %{clang-tidy} %{module}/std/cstdarg.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cstdarg.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.cstdarg.module
+// RUN: cat  %t.cstdarg.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cstdarg}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_24 | sort > %t.cstdarg.include
+// RUN: diff -u %t.cstdarg.module %t.cstdarg.include
+#if defined(TEST_24)
+#include <cstdarg>
+#endif
+// RUN: %{clang-tidy} %{module}/std/cstddef.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cstddef.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.cstddef.module
+// RUN: cat  %t.cstddef.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cstddef}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_25 | sort > %t.cstddef.include
+// RUN: diff -u %t.cstddef.module %t.cstddef.include
+#if defined(TEST_25)
+#include <cstddef>
+#endif
+// RUN: %{clang-tidy} %{module}/std/cstdio.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cstdio.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.cstdio.module
+// RUN: cat  %t.cstdio.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cstdio}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_26 | sort > %t.cstdio.include
+// RUN: diff -u %t.cstdio.module %t.cstdio.include
+#if defined(TEST_26)
+#include <cstdio>
+#endif
+// RUN: %{clang-tidy} %{module}/std/cstdlib.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cstdlib.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.cstdlib.module
+// RUN: cat  %t.cstdlib.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cstdlib}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_27 | sort > %t.cstdlib.include
+// RUN: diff -u %t.cstdlib.module %t.cstdlib.include
+#if defined(TEST_27)
+#include <cstdlib>
+#endif
+// RUN: %{clang-tidy} %{module}/std/cstring.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cstring.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.cstring.module
+// RUN: cat  %t.cstring.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cstring}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_28 | sort > %t.cstring.include
+// RUN: diff -u %t.cstring.module %t.cstring.include
+#if defined(TEST_28)
+#include <cstring>
+#endif
+// RUN: %{clang-tidy} %{module}/std/ctime.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: ctime.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.ctime.module
+// RUN: cat  %t.ctime.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: ctime}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_29 | sort > %t.ctime.include
+// RUN: diff -u %t.ctime.module %t.ctime.include
+#if defined(TEST_29)
+#include <ctime>
+#endif
+// RUN: %{clang-tidy} %{module}/std/cwctype.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cwctype.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.cwctype.module
+// RUN: cat  %t.cwctype.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: cwctype}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_31 | sort > %t.cwctype.include
+// RUN: diff -u %t.cwctype.module %t.cwctype.include
+#if defined(TEST_31)
+#include <cwctype>
+#endif
+// RUN: %{clang-tidy} %{module}/std/exception.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: exception.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.exception.module
+// RUN: cat  %t.exception.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: exception}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_33 | sort > %t.exception.include
+// RUN: diff -u %t.exception.module %t.exception.include
+#if defined(TEST_33)
+#include <exception>
+#endif
+// RUN: %{clang-tidy} %{module}/std/execution.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: execution.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.execution.module
+// RUN: cat  %t.execution.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: execution}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_34 | sort > %t.execution.include
+// RUN: diff -u %t.execution.module %t.execution.include
+#if defined(TEST_34)
+#include <execution>
+#endif
+// RUN: %{clang-tidy} %{module}/std/expected.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: expected.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.expected.module
+// RUN: cat  %t.expected.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: expected}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_35 | sort > %t.expected.include
+// RUN: diff -u %t.expected.module %t.expected.include
+#if defined(TEST_35)
+#include <expected>
+#endif
+// RUN: %{clang-tidy} %{module}/std/format.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: format.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.format.module
+// RUN: cat  %t.format.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: format}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_38 | sort > %t.format.include
+// RUN: diff -u %t.format.module %t.format.include
+#if defined(TEST_38)
+#include <format>
+#endif
+// RUN: %{clang-tidy} %{module}/std/future.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: future.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.future.module
+// RUN: cat  %t.future.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: future}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_39 | sort > %t.future.include
+// RUN: diff -u %t.future.module %t.future.include
+#if defined(TEST_39)
+#include <future>
+#endif
+// RUN: %{clang-tidy} %{module}/std/initializer_list.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: initializer_list.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.initializer_list.module
+// RUN: cat  %t.initializer_list.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: initializer_list}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_40 | sort > %t.initializer_list.include
+// RUN: diff -u %t.initializer_list.module %t.initializer_list.include
+#if defined(TEST_40)
+#include <initializer_list>
+#endif
+// RUN: %{clang-tidy} %{module}/std/iomanip.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: iomanip.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.iomanip.module
+// RUN: cat  %t.iomanip.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: iomanip} , {key: libcpp-header-exportable-declarations.SkipDeclarations, value: "std::operator<< std::operator>>"}  ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_42 | sort > %t.iomanip.include
+// RUN: diff -u %t.iomanip.module %t.iomanip.include
+#if defined(TEST_42)
+#include <iomanip>
+#endif
+// RUN: %{clang-tidy} %{module}/std/latch.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: latch.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.latch.module
+// RUN: cat  %t.latch.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: latch}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_43 | sort > %t.latch.include
+// RUN: diff -u %t.latch.module %t.latch.include
+#if defined(TEST_43)
+#include <latch>
+#endif
+// RUN: %{clang-tidy} %{module}/std/limits.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: limits.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.limits.module
+// RUN: cat  %t.limits.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: limits}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_44 | sort > %t.limits.include
+// RUN: diff -u %t.limits.module %t.limits.include
+#if defined(TEST_44)
+#include <limits>
+#endif
+// RUN: %{clang-tidy} %{module}/std/list.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: list.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.list.module
+// RUN: cat  %t.list.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: list}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_46 | sort > %t.list.include
+// RUN: diff -u %t.list.module %t.list.include
+#if defined(TEST_46)
+#include <list>
+#endif
+// RUN: %{clang-tidy} %{module}/std/new.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: new.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.new.module
+// RUN: cat  %t.new.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: new}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_49 | sort > %t.new.include
+// RUN: diff -u %t.new.module %t.new.include
+#if defined(TEST_49)
+#include <new>
+#endif
+// RUN: %{clang-tidy} %{module}/std/numbers.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: numbers.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.numbers.module
+// RUN: cat  %t.numbers.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: numbers}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_50 | sort > %t.numbers.include
+// RUN: diff -u %t.numbers.module %t.numbers.include
+#if defined(TEST_50)
+#include <numbers>
+#endif
+// RUN: %{clang-tidy} %{module}/std/numeric.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: numeric.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.numeric.module
+// RUN: cat  %t.numeric.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: numeric}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_51 | sort > %t.numeric.include
+// RUN: diff -u %t.numeric.module %t.numeric.include
+#if defined(TEST_51)
+#include <numeric>
+#endif
+// RUN: %{clang-tidy} %{module}/std/ratio.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: ratio.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.ratio.module
+// RUN: cat  %t.ratio.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: ratio}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_52 | sort > %t.ratio.include
+// RUN: diff -u %t.ratio.module %t.ratio.include
+#if defined(TEST_52)
+#include <ratio>
+#endif
+// RUN: %{clang-tidy} %{module}/std/semaphore.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: semaphore.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.semaphore.module
+// RUN: cat  %t.semaphore.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: semaphore}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_53 | sort > %t.semaphore.include
+// RUN: diff -u %t.semaphore.module %t.semaphore.include
+#if defined(TEST_53)
+#include <semaphore>
+#endif
+// RUN: %{clang-tidy} %{module}/std/shared_mutex.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: shared_mutex.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.shared_mutex.module
+// RUN: cat  %t.shared_mutex.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: shared_mutex}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_55 | sort > %t.shared_mutex.include
+// RUN: diff -u %t.shared_mutex.module %t.shared_mutex.include
+#if defined(TEST_55)
+#include <shared_mutex>
+#endif
+// RUN: %{clang-tidy} %{module}/std/source_location.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: source_location.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.source_location.module
+// RUN: cat  %t.source_location.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: source_location}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_56 | sort > %t.source_location.include
+// RUN: diff -u %t.source_location.module %t.source_location.include
+#if defined(TEST_56)
+#include <source_location>
+#endif
+// RUN: %{clang-tidy} %{module}/std/span.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: span.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.span.module
+// RUN: cat  %t.span.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: span}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_57 | sort > %t.span.include
+// RUN: diff -u %t.span.module %t.span.include
+#if defined(TEST_57)
+#include <span>
+#endif
+// RUN: %{clang-tidy} %{module}/std/stdexcept.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: stdexcept.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.stdexcept.module
+// RUN: cat  %t.stdexcept.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: stdexcept}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_61 | sort > %t.stdexcept.include
+// RUN: diff -u %t.stdexcept.module %t.stdexcept.include
+#if defined(TEST_61)
+#include <stdexcept>
+#endif
+// RUN: %{clang-tidy} %{module}/std/string.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: string.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.string.module
+// RUN: cat  %t.string.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: string} , {key: libcpp-header-exportable-declarations.SkipDeclarations, value: "std::allocator"}  ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_65 | sort > %t.string.include
+// RUN: diff -u %t.string.module %t.string.include
+#if defined(TEST_65)
+#include <string>
+#endif
+// RUN: %{clang-tidy} %{module}/std/string_view.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: string_view.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.string_view.module
+// RUN: cat  %t.string_view.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: string_view}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_67 | sort > %t.string_view.include
+// RUN: diff -u %t.string_view.module %t.string_view.include
+#if defined(TEST_67)
+#include <string_view>
+#endif
+// RUN: %{clang-tidy} %{module}/std/strstream.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: strstream.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.strstream.module
+// RUN: cat  %t.strstream.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: strstream}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_68 | sort > %t.strstream.include
+// RUN: diff -u %t.strstream.module %t.strstream.include
+#if defined(TEST_68)
+#include <strstream>
+#endif
+// RUN: %{clang-tidy} %{module}/std/typeindex.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: typeindex.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.typeindex.module
+// RUN: cat  %t.typeindex.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: typeindex}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_70 | sort > %t.typeindex.include
+// RUN: diff -u %t.typeindex.module %t.typeindex.include
+#if defined(TEST_70)
+#include <typeindex>
+#endif
+// RUN: %{clang-tidy} %{module}/std/typeinfo.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: typeinfo.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.typeinfo.module
+// RUN: cat  %t.typeinfo.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: typeinfo}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_71 | sort > %t.typeinfo.include
+// RUN: diff -u %t.typeinfo.module %t.typeinfo.include
+#if defined(TEST_71)
+#include <typeinfo>
+#endif
+// RUN: %{clang-tidy} %{module}/std/valarray.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: valarray.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.valarray.module
+// RUN: cat  %t.valarray.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: valarray} , {key: libcpp-header-exportable-declarations.SkipDeclarations, value: "std::sliceExpr"}  ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_73 | sort > %t.valarray.include
+// RUN: diff -u %t.valarray.module %t.valarray.include
+#if defined(TEST_73)
+#include <valarray>
+#endif
+// RUN: %{clang-tidy} %{module}/std/variant.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: variant.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.variant.module
+// RUN: cat  %t.variant.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: variant}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_74 | sort > %t.variant.include
+// RUN: diff -u %t.variant.module %t.variant.include
+#if defined(TEST_74)
+#include <variant>
+#endif
+// RUN: %{clang-tidy} %{module}/std/version.cppm --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: version.cppm} ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} | sort > %t.version.module
+// RUN: cat  %t.version.module >> %t.parts
+// RUN: %{clang-tidy} %s --checks='-*,libcpp-header-exportable-declarations' -config='{CheckOptions: [ {key: libcpp-header-exportable-declarations.Header, value: version}   ]}' --load=%{test-tools}/clang_tidy_checks/libcxx-tidy.plugin -- %{compile_flags} -DTEST_75 | sort > %t.version.include
+// RUN: diff -u %t.version.module %t.version.include
+#if defined(TEST_75)
+#include <version>
+#endif
+// RUN: sort -u -o %t.parts %t.parts
+// GENERATED-MARKER

libcxx/test/std/time/time.syn/formatter_tests.h

 #include "test_macros.h"
 
 #include <algorithm>
 #include <cassert>
 #include <iostream>
 #include <set>
 #include <string>
 #include <string_view>
+#include <iostream>
 
 #define STR(S) MAKE_STRING(CharT, S)
 #define SV(S) MAKE_STRING_VIEW(CharT, S)
 
 #ifndef TEST_HAS_NO_WIDE_CHARACTERS
 template <class CharT>
 using format_context = std::conditional_t<std::same_as<CharT, char>, std::format_context, std::wformat_context>;
 #else

libcxx/test/tools/clang_tidy_checks/header_exportable_declarations.hpp

+//===----------------------------------------------------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+
+#include "clang-tidy/ClangTidyCheck.h"
+
+namespace libcpp {
+class header_exportable_declarations : public clang::tidy::ClangTidyCheck {
+public:
+  header_exportable_declarations(llvm::StringRef, clang::tidy::ClangTidyContext*);
+  void registerMatchers(clang::ast_matchers::MatchFinder*) override;
+  void check(const clang::ast_matchers::MatchFinder::MatchResult&) override;
+
+private:
+  const std::optional<llvm::StringRef> header_;
+  const llvm::StringRef extra_header_;
+  std::set<std::string> decls_;
+};
+} // namespace libcpp

libcxx/test/tools/clang_tidy_checks/header_exportable_declarations.cpp

+//===----------------------------------------------------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+
+#include "clang-tidy/ClangTidyCheck.h"
+#include "clang-tidy/ClangTidyModuleRegistry.h"
+
+#include "header_exportable_declarations.hpp"
+
+#include <iostream>
+#include <iterator>
+#include <ranges>
+
+namespace libcpp {
+header_exportable_declarations::header_exportable_declarations(
+    llvm::StringRef name, clang::tidy::ClangTidyContext* context)
+    : clang::tidy::ClangTidyCheck(name, context),
+      header_(Options.get("Header")),
+      extra_header_(Options.get("ExtraHeader", "")) {
+  if (!header_)
+    llvm::errs() << "No header is provided.\n";
+
+  // Header -> name
+  // Module -> name
+  //
+  // SkipDeclaration -> space separated list declarationst to ignore
+  //   They will be added to decls_. Note typical separators may be part of the
+  //   identifier.
+
+  std::optional<llvm::StringRef> list = Options.get("SkipDeclarations");
+  if (list)
+    for (auto decl : std::views::split(*list, ' ')) {
+      std::string s;
+      std::ranges::copy(decl, std::back_inserter(s)); // use range based constructor
+      decls_.emplace(std::move(s));
+    }
+
+  list = Options.get("ExtraDeclarations");
+  if (list)
+    for (auto decl : std::views::split(*list, ' '))
+      std::cout << "using " << std::string_view{decl.data(), decl.size()} << ";\n";
+
+  if (std::optional<llvm::StringRef> file = Options.get("StaticAssertFile"); file)
+    static_assert_file_.open(std::string{*file}, std::ios_base::out | std::ios_base::app);
+}
+
+void header_exportable_declarations::registerMatchers(clang::ast_matchers::MatchFinder* finder) {
+  // there are no public names in the Standard starting with an underscore, so no need to check the strict rules
+  using namespace clang::ast_matchers;
+
+  bool is_module = header_->endswith(".cppm");
+  assert(!(is_module && static_assert_file_.is_open()) && "static asserts are only generated from headers");
+
+  if (!is_module)
+    finder->addMatcher(
+        namedDecl(
+            // Looks at the common locations where headers store their data
+            // * header
+            // * __header/*.h
+            // * __fwd/header.h
+            anyOf(isExpansionInFileMatching(("v1/__" + *header_ + "/").str()),
+                  isExpansionInFileMatching(extra_header_),
+                  isExpansionInFileMatching(("v1/__fwd/" + *header_ + "\\.h$").str()),
+                  isExpansionInFileMatching(("v1/" + *header_ + "$").str())),
+            unless(hasAncestor(friendDecl())))
+            .bind("header_exportable_declarations"),
+        this);
+  else
+    finder->addMatcher(namedDecl(isExpansionInFileMatching(*header_)).bind("header_exportable_declarations"), this);
+}
+
+/// Returns the qualified name of a declaration.
+///
+/// There is a small issue with qualified names. Typically the name returned is
+/// in the namespace \c std instead of the namespace \c std::__1. Except when a
+/// name is declared both in the namespace \c std and in the namespace
+/// \c std::__1. In that case the returned value will adjust the name to use
+/// the namespace \c std.
+///
+/// The reason this happens is due to some parts of libc++ using
+/// \code namespace std \endcode instead of
+/// \code _LIBCPP_BEGIN_NAMESPACE_STD \endcode
+/// Some examples
+/// * cstddef has bitwise operators for the type \c byte
+/// * exception has equality operators for the type \c exception_ptr
+/// * initializer_list has the functions \c begin and \c end
+///
+/// TODO is this an issue in libc++?
+static std::string get_qualified_name(const clang::NamedDecl& decl) {
+  std::string result = decl.getQualifiedNameAsString();
+
+  if (result.starts_with("std::__1::"))
+    result.erase(5, 5);
+
+  return result;
+}
+
+static bool is_viable_declaration(const clang::NamedDecl* decl) {
+  // Declarations nested in records are automatically exported with the record itself.
+  if (!decl->getDeclContext()->isNamespace())
+    return false;
+
+  // Declarations that are a subobject of a friend Declaration are automatically exported with the record itself.
+  if (decl->getFriendObjectKind() != clang::Decl::FOK_None)
+    return false;
+
+  // *** Function declarations ***
+
+  if (clang::CXXMethodDecl::classof(decl))
+    return false;
+
+  if (clang::CXXDeductionGuideDecl::classof(decl))
+    return false;
+
+  if (clang::FunctionDecl::classof(decl))
+    return true;
+
+  if (clang::CXXConstructorDecl::classof(decl))
+    return false;
+
+  // implicit constructors disallowed
+  if (const auto* r = llvm::dyn_cast_or_null<clang::RecordDecl>(decl))
+    return !r->isLambda() && !r->isImplicit();
+
+    // *** Record declarations ***
+
+#if 0
+  // Enumerate declarations
+  if (clang::EnumDecl::classof(decl))
+    return true;
+
+  // Variable declarations
+  if (clang::VarDecl::classof(decl))
+    return true;
+
+  // Concept declarations
+  if (clang::ConceptDecl::classof(decl))
+    return true;
+
+  // Using declarations
+  if (clang::TypedefNameDecl::classof(decl))
+    return true;
+
+  if (clang::UsingDecl::classof(decl))
+    return true;
+
+  return false;
+#else
+  return llvm::isa<clang::EnumDecl, clang::VarDecl, clang::ConceptDecl, clang::TypedefNameDecl, clang::UsingDecl>(decl);
+#endif
+}
+
+/// Returns the name is a reserved name.
+///
+/// This test misses 2 candidates which are not used in libc++
+/// * any identifier with two underscores not at the start
+/// * a name with a leading underscore in the global namespace
+bool is_reserved_name(const std::string& name) {
+  std::size_t pos = name.find("::_");
+  if (pos == std::string::npos)
+    return false;
+
+  if (pos + 3 > name.size())
+    return false;
+
+  return name[pos + 3] == '_' || std::isupper(name[pos + 3]);
+}
+
+void header_exportable_declarations::check(const clang::ast_matchers::MatchFinder::MatchResult& result) {
+  if (const auto* decl = result.Nodes.getNodeAs<clang::NamedDecl>("header_exportable_declarations"); decl != nullptr) {
+    if (!is_viable_declaration(decl))
+      return;
+
+    std::string name = get_qualified_name(*decl);
+    if (is_reserved_name(name))
+      return;
+
+    if (decls_.contains(name))
+      return;
+
+    std::cout << "using " << std::string{name} << ";\n";
+    decls_.insert(name);
+  }
+}
+
+} // namespace libcpp

libcxx/utils/libcxx/test/params.py

_warningFlags = [
   '-Wunused-template',
   '-Wno-unused-command-line-argument',
   '-Wno-attributes',
   '-Wno-pessimizing-move',
   '-Wno-c++11-extensions',
   '-Wno-noexcept-type',
   '-Wno-aligned-allocation-unavailable',
   '-Wno-atomic-alignment',
+  '-Wno-reserved-module-identifier',
 
   # GCC warns about places where we might want to add sized allocation/deallocation
   # functions, but we know better what we're doing/testing in the test suite.
   '-Wno-sized-deallocation',
 
   # Turn off warnings about user-defined literals with reserved suffixes. Those are
   # just noise since we are testing the Standard Library itself.
   '-Wno-literal-suffix', # GCC
fallbacks = {
     'c++20': 'c++2a',
   }
   if hasCompileFlag(cfg, '-std='+std):
     return '-std='+std
   if std in fallbacks and hasCompileFlag(cfg, '-std='+fallbacks[std]):
     return '-std='+fallbacks[std]
   return None
 
+_allModules = ['none', 'clang', 'std', 'std.compat']
+def getModuleFlag(cfg, enable_modules):
+  # Originaly the flag was a Boolean, this maps the original Boolean values
+  # to the new enumerate values.
+  fallbacks = {
+    'none': 'False',
+    'clang': 'True',
+  }
+  if enable_modules in _allModules:
+    return enable_modules
+  if enable_modules in fallbacks:
+    return fallbacks[enable_modules]
+  return None
+
+
 DEFAULT_PARAMETERS = [
   Parameter(name='target_triple', type=str,
             help="The target triple to compile the test suite for. This must be "
                  "compatible with the target that the tests will be run on.",
             actions=lambda triple: filter(None, [
               AddFeature('target={}'.format(triple)),
               AddFlagIfSupported('--target={}'.format(triple)),
               AddSubstitution('%{triple}', triple)
             ])),
 
   Parameter(name='std', choices=_allStandards, type=str,
             help="The version of the standard to compile the test suite with.",
             default=lambda cfg: next(s for s in reversed(_allStandards) if getStdFlag(cfg, s)),
             actions=lambda std: [
               AddFeature(std),
               AddSubstitution('%{cxx_std}', re.sub('\+','x', std)),
               AddCompileFlag(lambda cfg: getStdFlag(cfg, std)),
             ]),
 
-  Parameter(name='enable_modules', choices=[True, False], type=bool, default=False,
-            help="Whether to build the test suite with Clang modules enabled.",
-            actions=lambda modules: [
+  Parameter(name='enable_modules', choices=_allModules + ['True', 'False'], type=str,
+            help="Whether to build the test suite with modules enabled. Select "
+                 "Clang for Clang modules and c++ for C++ Standard modules",
+            default=lambda cfg: next(s for s in _allModules if getModuleFlag(cfg, s)),
+            actions=lambda enable_modules: [
               AddFeature('modules-build'),
               AddCompileFlag('-fmodules'),
               AddCompileFlag('-fcxx-modules'), # AppleClang disregards -fmodules entirely when compiling C++. This enables modules for C++.
-            ] if modules else []),
+            ] if enable_modules == "clang" or enable_modules == "True" else [
+              AddFeature('use_module_std'),
+              AddCompileFlag('-DTEST_USE_MODULE'),
+              AddCompileFlag('-DTEST_USE_MODULE_STD'),
+              AddCompileFlag(lambda cfg: '-fprebuilt-module-path=' + os.path.join(cfg.test_exec_root, '__config_module__/CMakeFiles/std.dir')),
+              AddLinkFlag(lambda cfg: os.path.join(cfg.test_exec_root, '__config_module__/libstd.a')),
+              AddModule(),
+            ] if enable_modules == "std" else [
+              AddFeature('use_module_std.compat'),
+              AddCompileFlag('-DTEST_USE_MODULE'),
+              AddCompileFlag("-DTEST_USE_MODULE_STD_COMPAT"),
+              AddCompileFlag(lambda cfg: '-fprebuilt-module-path=' + os.path.join(cfg.test_exec_root, '__config_module__/CMakeFiles/std.compat.dir')),
+              AddLinkFlag(lambda cfg: os.path.join(cfg.test_exec_root, '__config_module__/libstd.compat.a')),
+              AddLinkFlag(lambda cfg: os.path.join(cfg.test_exec_root, '__config_module__/libstd.a')),
+              AddModule(),
+            ] if enable_modules == "std.compat" else []),
 
   Parameter(name='enable_modules_lsv', choices=[True, False], type=bool, default=False,
           help="Whether to enable Local Submodule Visibility in the Modules build.",
           actions=lambda lsv: [
             AddCompileFlag('-Xclang -fmodules-local-submodule-visibility'),
           ] if lsv else []),
 
   Parameter(name='enable_exceptions', choices=[True, False], type=bool, default=True,

libcxx/utils/use_modules_in_test.py

+#!/usr/bin/env python
+# ===------------------------------------------------------------------------------===##
+#
+# Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+# See https://llvm.org/LICENSE.txt for license information.
+# SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+#
+# ===------------------------------------------------------------------------------===##
+
+# This script converts a regular test to a test using modules.
+#
+# Note this script is mainly used to test the patch https://reviews.llvm.org/D144994
+# It has not been decided that libc++ will use this approach for modules. Therefore
+# the transformation uses a simple algorithm and generates not the prettiest output.
+#
+# The main goal is to quickly convert a batch of tests to the use modules to test
+# whether the std module works for the headers that have been converted. The
+# script can't be executed twice on the same header, so it's intended this
+# script is executed on a not-yet converted code base.
+#
+# Usage
+#   use_modules_in_test.py <file>
+#
+# If the file contains at least one of the headers that is available in the std module
+# it will add a guarded import statement. All headers that are in the module std will
+# be included conditionally.
+
+import re
+import sys
+from dataclasses import dataclass, field
+
+
+HEADERS_MODULE_STD = [
+    "algorithm",
+    "any",
+    "array",
+    "atomic",
+    "barrier",
+    "bit",
+    "bitset",
+    "cctype",
+    "cfenv",
+    "charconv",
+    "chrono",
+    "cinttypes",
+    "climits",
+    "clocale",
+    "cmath",
+    "codecvt",
+    "compare",
+    "complex",
+    "concepts",
+    "coroutine",
+    "csetjmp",
+    "csignal",
+    "cstdarg",
+    "cstddef",
+    "cstdio",
+    "cstdlib",
+    "cstlib",
+    "cstring",
+    "ctime",
+    "ctype.h",
+    "cuchar",
+    "cwchar",
+    "cwctype",
+    "deque",
+    "exception",
+    "execution",
+    "expected",
+    "fenv.h",
+    "filesystem",
+    "flat_map",
+    "flat_set",
+    "float.h",
+    "format",
+    "forward_list",
+    "fstream",
+    "functional",
+    "generator",
+    "initializer_list",
+    "inttypes.h",
+    "iomanip",
+    "ios",
+    "iostream",
+    "istream",
+    "iterator",
+    "latch",
+    "limits",
+    "list",
+    "locale",
+    "locale.h",
+    "map",
+    "math.h",
+    "mdspan",
+    "memory",
+    "memory_resource",
+    "mutex",
+    "new",
+    "numbers",
+    "numeric",
+    "optional",
+    "ostream",
+    "print",
+    "queue",
+    "random",
+    "ranges",
+    "ratio",
+    "regex",
+    "scoped_allocator",
+    "semaphore",
+    "set",
+    "setjmp.h",
+    "shared_mutex",
+    "signal.h",
+    "source_location",
+    "span",
+    "spanstream",
+    "sstream",
+    "stack",
+    "stacktrace",
+    "stdarg.h",
+    "stddef.h",
+    "stdexcept",
+    "stdfloat",
+    "stdint.h",
+    "stdio.h",
+    "stdlib.h",
+    "stop_token",
+    "streambuf",
+    "string",
+    "string.h",
+    "string_view",
+    "strstream",
+    "syncstream",
+    "system_error",
+    "thread",
+    "time.h",
+    "tuple",
+    "type_traits",
+    "typeindex",
+    "typeinfo",
+    "uchar.h",
+    "unordered_map",
+    "unordered_set",
+    "utility",
+    "valarray",
+    "variant",
+    "vector",
+    "wchar.h",
+    "wctype.h",
+]
+
+# Tests that need to be marked as XFAIL for various reasons
+#
+# For example is_literal type has been removed from C++20, the tests reenables it using
+# test specific compile flags. This is not supported by modules.
+UNSUPPORTED_TESTS = [
+    "libcxx/test/std/utilities/meta/meta.unary/meta.unary.prop/is_literal_type.pass.cpp",
+]
+# Due to a Clang limitation the import needs to be after all includes.
+# Therefore non-system includes are parsed too.
+INCLUDE_REGEX = re.compile(r"^\s*#\s*include\s*[\"<](?P<include>[^\">]+)[\">]")
+
+# Ignore tests that issue deprecation warnings.
+#
+# An alternative would be just to remove this option, this would be possible
+# when the script is only used to convert in the CI.
+DISABLE_DEPRECATION_WARNINGS_REGEXES = [
+    re.compile(
+        r"//\s*ADDITIONAL_COMPILE_FLAGS:.*-D_LIBCPP_DISABLE_DEPRECATION_WARNINGS"
+    ),
+    re.compile(r"//\s*ADDITIONAL_COMPILE_FLAGS:.*-D_LIBCPP_ENABLE_ASSERTIONS"),
+    re.compile(r"//\s*ADDITIONAL_COMPILE_FLAGS:.*-D_LIBCPP_HAS_NO_UNICODE"),
+    re.compile(r"//\s*ADDITIONAL_COMPILE_FLAGS:.*-fsized-deallocation"),
+]
+
+
+# Marks a test as unsupported.
+#
+# Since the script runs on a not yet converted set of tests the script does not
+# test existing markers. In the future it should look at:
+# - tests with (XFAIL|UNSUPPORTED) use_module_std(|.compat)
+# - tests with an import statement
+def is_unsupported(filename: str) -> bool:
+    file = open(filename)
+    lines = file.readlines()
+    file.close()
+
+    for index, line in enumerate(lines):
+        for re in DISABLE_DEPRECATION_WARNINGS_REGEXES:
+            if re.search(line):
+                lines.insert(
+                    index + 1, "// UNSUPPORTED: use_module_std, use_module_std.compat\n"
+                )
+
+                file = open(filename, "w")
+                for line in lines:
+                    file.write(line)
+
+                return True
+    return False
+
+
+@dataclass
+class Include:
+    line: int = -1
+    is_system_header: bool = False
+
+
+def gather_includes(lines: list[str]) -> list[Include]:
+    result = list()
+    for index, line in enumerate(lines):
+        if m := INCLUDE_REGEX.match(line):
+            result.append(Include(index, m.group("include") in HEADERS_MODULE_STD))
+    return result
+
+
+def insert_imports(lines: list[str], includes: list[Include]) -> None:
+    assert len(lines)
+    assert len(includes)
+
+    # The imports are always added after the last header
+    # Note this fails when a test has an #include in the middle of the file
+
+    line = includes[-1].line + 1
+
+    imports = list()
+    lines.insert(
+        line,
+        """
+#if defined(TEST_USE_MODULE_STD)
+import std;
+#elif defined(TEST_USE_MODULE_STD_COMPAT)
+import std.compat;
+#endif
+""",
+    )
+
+
+def make_modularized_headers_conditionally(
+    lines: list[str], includes: list[Include]
+) -> None:
+    assert len(lines)
+    assert len(includes)
+
+    inside = False
+    # Iterate backwards so the line numbers of the includes remain stable.
+    for include in reversed(includes):
+        if include.is_system_header:
+            if not inside:
+                lines.insert(include.line + 1, "#endif // TEST_USE_MODULE\n")
+                inside = True
+        else:
+            if inside:
+                lines.insert(include.line + 1, "#ifdef TEST_USE_MODULE\n")
+                inside = False
+
+    # Add the ifdef for the first entry if needed.
+    if inside:
+        for include in includes:
+            if include.is_system_header:
+                lines.insert(include.line, "#ifdef TEST_USE_MODULE\n")
+                return
+        assert False
+
+
+def process_supported(filename: str) -> None:
+    import_written = False
+
+    file = open(filename)
+    lines = file.readlines()
+    file.close()
+    includes = gather_includes(lines)
+    # for include in includes:
+    #    print(f"{include.is_system_header} {include.line:4}: {lines[include.line]}")
+    if len(includes) == 0:
+        return
+
+    # TODO Once Clang works properly with includes after imports this import can be
+    # at the top of the file.
+    insert_imports(lines, includes)
+    # for line in lines:
+    #    print(line)
+
+    make_modularized_headers_conditionally(lines, includes)
+    # for line in lines:
+    #    print(line)
+
+    file = open(filename, "w")
+    for line in lines:
+        file.write(line)
+
+
+def process(filename: str) -> None:
+    if not is_unsupported(filename):
+        process_supported(filename)
+
+
+if __name__ == "__main__":
+    if len(sys.argv) == 2:
+        process(sys.argv[1])