This is an archive of the discontinued LLVM Phabricator instance.

[Support] Don't include VirtualFileSystem.h in CommandLine.h
ClosedPublic

Authored by thakis on Apr 21 2021, 7:02 AM.

Details

Summary

CommandLine.h is indirectly included in ~50% of TUs when building
clang, and VirtualFileSystem.h is large.

(Already remarked by jhenderson on D70769.)

No behavior change.

Diff Detail

Event Timeline

thakis created this revision.Apr 21 2021, 7:02 AM
thakis requested review of this revision.Apr 21 2021, 7:02 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 21 2021, 7:02 AM
hans accepted this revision.Apr 21 2021, 7:06 AM

nice!

llvm/lib/CodeGen/GlobalISel/CSEInfo.cpp
13

sort looks off

llvm/lib/Support/WithColor.cpp
10

sort

This revision is now accepted and ready to land.Apr 21 2021, 7:06 AM
thakis marked 2 inline comments as done.Apr 21 2021, 7:19 AM

Thanks!

This revision was landed with ongoing or failed builds.Apr 21 2021, 7:19 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptApr 21 2021, 7:19 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript