This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Granularize <iterator> includes
ClosedPublic

Authored by philnik on Jun 9 2022, 2:42 PM.

Details

Reviewers
ldionne
Mordante
var-const
Group Reviewers
Restricted Project
Commits
rG3cd4531b9ba4: [libc++] Granularize <iterator> includes

Diff Detail

Event Timeline

philnik created this revision.Jun 9 2022, 2:42 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 9 2022, 2:42 PM
Herald added a subscriber: wenlei. · View Herald Transcript
philnik requested review of this revision.Jun 9 2022, 2:42 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 9 2022, 2:42 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
philnik retitled this revision from [libc++] Granulasize <iterator> includes to [libc++] Granularize <iterator> includes.Jun 10 2022, 7:27 AM
ldionne accepted this revision.Jun 10 2022, 8:40 AM

Let's follow up with the removal of <iterator> before LLVM 15, since we are already removing several transitive includes. If we are going to cause some work to downstream users, we might as well try to get the most out of it (and try not to break them at every release if we can help it).

This revision is now accepted and ready to land.Jun 10 2022, 8:40 AM
philnik updated this revision to Diff 435977.Jun 10 2022, 10:53 AM
  • Try to fix CI
This revision was automatically updated to reflect the committed changes.