This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Remove unused <iosfwd> include from <__debug>
ClosedPublic

Authored by ldionne on Apr 3 2022, 9:41 AM.

Details

Reviewers
None
Group Reviewers
Restricted Project
Commits
rGe70533ae6c57: [libc++] Remove unused <iosfwd> include from <__debug>

Diff Detail

Event Timeline

ldionne created this revision.Apr 3 2022, 9:41 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 3 2022, 9:41 AM
ldionne requested review of this revision.Apr 3 2022, 9:41 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 3 2022, 9:41 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
philnik added inline comments.
libcxx/include/module.modulemap
622

Why are you exporting iosfwd from iterator?

ldionne added inline comments.Apr 3 2022, 10:07 AM
libcxx/include/module.modulemap
622

It is needed because std::ostreambuf_iterator & friends have a default template argument of std::char_traits. I could also try to use something more restricted in scope like exporting char_traits directly from __iterator/ostreambuf_iterator.h. I'll take a look.

ldionne updated this revision to Diff 420069.Apr 3 2022, 10:51 AM

Try more localized fix

ldionne accepted this revision as: Restricted Project.Apr 3 2022, 1:15 PM
This revision was not accepted when it landed; it landed in state Needs Review.Apr 3 2022, 1:16 PM
This revision was automatically updated to reflect the committed changes.