This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Fixes the Clang modular build.
ClosedPublic

Authored by Mordante on Jan 26 2023, 11:03 PM.

Details

Reviewers
ldionne
Group Reviewers
Restricted Project
Commits
rGa7f8ddc4e2e3: [libc++] Fixes the Clang modular build.
Summary

The patch
430b397f6715d4d2f2569716c93fcd6292ace3dc ("[libc++] Granularize <type_traits> includes in <iterator>") missed some exports in the modulemap with became apparent in the reverting commit 12cb1cb3720d.

This fixes the issue by updating the modulemap. Note that the Clang ICE is not fixed by this change.

Diff Detail

Event Timeline

Mordante created this revision.Jan 26 2023, 11:03 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 26 2023, 11:03 PM
Mordante requested review of this revision.Jan 26 2023, 11:03 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptJan 26 2023, 11:03 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Mordante added inline comments.Jan 26 2023, 11:05 PM
libcxx/include/module.modulemap.in
1421

Note this looks wrong from libc++'s perspective. When this "solves" the issue we need to investigate why it fixes it; I think the export belongs to a different header.

Mordante updated this revision to Diff 492648.Jan 26 2023, 11:08 PM

Fix CI run instead of libc++.

Mordante updated this revision to Diff 492779.Jan 27 2023, 8:45 AM

Rebased and improve fixes.

Mordante updated this revision to Diff 492828.Jan 27 2023, 9:54 AM

Finalized patch.

Mordante updated this revision to Diff 492834.Jan 27 2023, 10:00 AM

Removes uninteded include.

Mordante updated this revision to Diff 492999.Jan 28 2023, 6:39 AM

Undo CI changes.

Mordante retitled this revision from [DRAFT][libc++] Fixes the Clang modular build. to [libc++] Fixes the Clang modular build..Jan 28 2023, 6:51 AM
Mordante edited the summary of this revision. (Show Details)

https://buildkite.com/llvm-project/libcxx-ci/builds/18148 shows this patch fixes the Clang modular build.

ldionne accepted this revision.Jan 30 2023, 1:24 PM
This revision is now accepted and ready to land.Jan 30 2023, 1:24 PM
This revision was landed with ongoing or failed builds.Jan 31 2023, 10:20 AM
This revision was automatically updated to reflect the committed changes.