This is an archive of the discontinued LLVM Phabricator instance.

[NFC] Reduce include files dependency.
ClosedPublic

Authored by dfukalov on Dec 2 2020, 9:07 AM.

Details

Summary
  1. Removed #include "...AliasAnalysis.h" in other headers and modules.
  2. Cleaned up includes in AliasAnalysis.h.

Diff Detail

Event Timeline

dfukalov created this revision.Dec 2 2020, 9:07 AM
Herald added a project: Restricted Project. · View Herald Transcript
dfukalov requested review of this revision.Dec 2 2020, 9:07 AM
bmahjour removed a subscriber: bmahjour.Dec 2 2020, 9:27 AM

Transform/IPO changes LGTM.

Thanks for working on this!

llvm/include/llvm/Analysis/AliasAnalysis.h
59

(style) ideally we try to keep these sorted

dfukalov updated this revision to Diff 309027.Dec 2 2020, 12:07 PM

Addressed comment, fixed polly build, fixed clang-tidy comments.

dfukalov marked an inline comment as done.Dec 2 2020, 2:14 PM
RKSimon accepted this revision.Dec 3 2020, 6:02 AM

LGTM - cheers!

This revision is now accepted and ready to land.Dec 3 2020, 6:02 AM
This revision was landed with ongoing or failed builds.Dec 3 2020, 7:25 AM
This revision was automatically updated to reflect the committed changes.