Skip to content

Commit 064f68f

Browse files
committed
Don't re-export top-level modules
In https://reviews.llvm.org/D119036 we fixed some of the infrastructure by removing the textual keyword. The underlying issue of PR50592 was that clang can re-export only submodules but under some conditions we needed to re-export the standalone module std_config via std. This patch provides a better fix to the symptom D119036 fixed. Differential revision: https://reviews.llvm.org/D142805
1 parent fd67e95 commit 064f68f

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

libcxx/include/module.modulemap.in

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,11 @@
22
// since __config may be included from C headers which may create an
33
// include cycle.
44
module std_config [system] [extern_c] {
5-
header "__config"
5+
textual header "__config"
66
export *
77
}
88

99
module std [system] {
10-
export std_config
1110
// FIXME: The standard does not require that each of these submodules
1211
// re-exports its imported modules. We should provide an alternative form of
1312
// export that issues a warning if a name from the submodule is used, and

0 commit comments

Comments
 (0)