This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Granularize <functional> includes
ClosedPublic

Authored by philnik on Apr 17 2022, 7:02 AM.

Details

Reviewers
Mordante
var-const
Group Reviewers
Restricted Project
Commits
rGfaef447e72a5: [libc++] Granularize <functional> includes

Diff Detail

Event Timeline

philnik created this revision.Apr 17 2022, 7:02 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 17 2022, 7:02 AM
Herald added a subscriber: miyuki. · View Herald Transcript
philnik requested review of this revision.Apr 17 2022, 7:02 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 17 2022, 7:02 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Mordante accepted this revision.Apr 20 2022, 11:42 AM

Nice cleanup, LGTM!

libcxx/include/experimental/coroutine
52

I wonder do we need to update this header? It's slated for removal.

libcxx/include/numeric
150

Interesting to see how many headers include this header and don't use it.

This revision is now accepted and ready to land.Apr 20 2022, 11:42 AM
This revision was automatically updated to reflect the committed changes.