This is an archive of the discontinued LLVM Phabricator instance.

Cleanup includes for LLVMBitsream
ClosedPublic

Authored by serge-sans-paille on Jan 27 2022, 8:12 AM.

Details

Summary

This is a very small library, so only a few changes and very little gain.

$ clang++ -E -Iinclude -I../llvm/include ../llvm/lib/Bitstream/Reader/*.cpp -std=c++14 -fno-rtti -fno-exceptions | wc -l
before: 65777
after: 64997

Related Discourse thread: https://llvm.discourse.group/t/include-what-you-use-include-cleanup

Diff Detail

Event Timeline

serge-sans-paille requested review of this revision.Jan 27 2022, 8:12 AM
serge-sans-paille created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptJan 27 2022, 8:12 AM
RKSimon accepted this revision.Jan 27 2022, 9:17 AM

LGTM - cheers

This revision is now accepted and ready to land.Jan 27 2022, 9:17 AM
This revision was landed with ongoing or failed builds.Jan 28 2022, 2:51 AM
This revision was automatically updated to reflect the committed changes.