This is an archive of the discontinued LLVM Phabricator instance.

Cleanup LLVMObject headers
ClosedPublic

Authored by serge-sans-paille on Feb 10 2022, 9:53 AM.

Details

Summary

Most notably,

llvm/Object/Binary.h no longer includes llvm/Support/MemoryBuffer.h
llvm/Object/MachOUniversal*.h no longer include llvm/Object/Archive.h
llvm/Object/TapiUniversal.h no longer includes llvm/Object/TapiFile.h

llvm-project preprocessed size:
before: 1068185081
after: 1068324320

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

Diff Detail

Event Timeline

serge-sans-paille requested review of this revision.Feb 10 2022, 9:53 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 10 2022, 9:53 AM
ormris removed a subscriber: ormris.Feb 10 2022, 9:54 AM
MaskRay accepted this revision.Feb 10 2022, 9:55 AM

Thanks!

This revision is now accepted and ready to land.Feb 10 2022, 9:55 AM
This revision was landed with ongoing or failed builds.Feb 10 2022, 12:14 PM
This revision was automatically updated to reflect the committed changes.

Thanks @MaskRay for the tireless reviews o/