This is an archive of the discontinued LLVM Phabricator instance.

Cleanup codegen includes
ClosedPublic

Authored by serge-sans-paille on Mar 15 2022, 3:03 AM.

Details

Summary

This is a (fixed) recommit of https://reviews.llvm.org/D121169

after: 1061034926
before: 1063332844

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

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptMar 15 2022, 3:03 AM
serge-sans-paille requested review of this revision.Mar 15 2022, 3:03 AM

Rebased on main + fix windows build

MaskRay accepted this revision.Mar 15 2022, 10:20 AM

For a fix, you don't need to create a Differential, but the commit message usually needs to mention what has been fixed.

This revision is now accepted and ready to land.Mar 15 2022, 10:20 AM

I wanted the pre-commit validation ;-)

You could reopen the original Differential and upload a new diff. That should trigger build bots again.

This revision was landed with ongoing or failed builds.Mar 16 2022, 12:43 AM
This revision was automatically updated to reflect the committed changes.