This is an archive of the discontinued LLVM Phabricator instance.

Cleanup includes: TransformsUtils
ClosedPublic

Authored by serge-sans-paille on Mar 1 2022, 9:29 AM.

Details

Summary

Estimation on the impact on preprocessor output:
before: 1065307662
after: 1064800684

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.Mar 1 2022, 9:29 AM
Herald added a reviewer: sstefan1. · View Herald Transcript
Herald added a reviewer: baziotis. · View Herald Transcript
Herald added a project: Restricted Project. · View Herald Transcript
lenary accepted this revision.Mar 1 2022, 9:54 AM

LGTM with one tiny nit.

llvm/lib/Analysis/CallGraphSCCPass.cpp
42

nit: Unnecessary change.

This revision is now accepted and ready to land.Mar 1 2022, 9:54 AM
This revision was automatically updated to reflect the committed changes.