This is an archive of the discontinued LLVM Phabricator instance.

Don't re-export top-level modules.
ClosedPublic

Authored by v.g.vassilev on Jan 28 2023, 4:22 AM.

Details

Reviewers
ldionne
ChuanqiXu
Group Reviewers
Restricted Project
Commits
rG064f68f7c451: Don't re-export top-level modules
Summary

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.

Diff Detail

Event Timeline

v.g.vassilev created this revision.Jan 28 2023, 4:22 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 28 2023, 4:22 AM
ChuanqiXu accepted this revision.Jan 30 2023, 9:14 PM
This revision is now accepted and ready to land.Jan 30 2023, 9:14 PM
ldionne accepted this revision.Feb 3 2023, 3:07 PM
This revision was landed with ongoing or failed builds.Feb 6 2023, 9:35 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptFeb 6 2023, 9:35 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript