This is an archive of the discontinued LLVM Phabricator instance.

[C++20][Modules][6/8] Record direct module imports.
ClosedPublic

Authored by iains on Jan 31 2022, 2:39 AM.

Details

Summary

This is a small cache to avoid having to check both Exports and
Imports.

Diff Detail

Event Timeline

iains created this revision.Jan 31 2022, 2:39 AM
iains updated this revision to Diff 404485.Jan 31 2022, 4:45 AM

Rebased onto dependent patch after changes

iains updated this revision to Diff 405596.Feb 3 2022, 5:39 AM

rebased onto import state machine.

iains updated this revision to Diff 408764.Feb 15 2022, 1:29 AM

Rebased onto other modules work.

iains published this revision for review.Feb 15 2022, 8:33 AM

patch 6 of 8 to implement basic C++20 module partition support.

Herald added a project: Restricted Project. · View Herald TranscriptFeb 15 2022, 8:33 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
iains updated this revision to Diff 409205.Feb 16 2022, 4:28 AM

rebased onto changes in parent patches.

iains retitled this revision from [C++20][Modules] Record direct module imports. to [C++20][Modules][6/8] Record direct module imports..Feb 16 2022, 4:28 AM

hmmm, I don't find this is used in following patch. Oversight?

ChuanqiXu accepted this revision.Feb 17 2022, 1:39 AM

Sorry, I took an oversight.

This revision is now accepted and ready to land.Feb 17 2022, 1:39 AM
iains updated this revision to Diff 409645.Feb 17 2022, 7:18 AM

rebased onto parent patch changes

iains updated this revision to Diff 410404.Feb 21 2022, 3:19 PM

rebased

iains updated this revision to Diff 410528.Feb 22 2022, 6:58 AM

rebased.

iains updated this revision to Diff 411545.Feb 25 2022, 4:00 PM

rebased

This revision was landed with ongoing or failed builds.Feb 27 2022, 2:07 AM
This revision was automatically updated to reflect the committed changes.