This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Put __memory headers in submodules
AbandonedPublic

Authored by teemperor on Jun 3 2021, 1:33 AM.

Details

Reviewers
ldionne
Group Reviewers
Restricted Project
Summary

Currently all the headers in __memory are not submodules of std but normal headers that get duplicated everywhere.

This patch marks all of them that can be parsed on their own as submodules so we only need one copy of them around.

This also means Clang no longer has to deal with making sense of all the duplicated declarations which leads to bogus
module build failures as seen in D103369.

Diff Detail

Event Timeline

teemperor requested review of this revision.Jun 3 2021, 1:33 AM
teemperor created this revision.
Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 3 2021, 1:33 AM
teemperor added a subscriber: vsk.Jun 3 2021, 1:36 AM

That should fix the build failures that caused the revert of D103369.

I believe this is effectively a subset of @cjdb's D103551, and D103551 will be the better (completer) fix, so this one should be abandoned.

teemperor abandoned this revision.Jun 3 2021, 6:18 AM

You are right, I wasn't aware of D103551. Thanks!

Abandoning this in favour of D103551