This is an archive of the discontinued LLVM Phabricator instance.

[llvm] Reduce ComplexDeinterleavingPass.h includes
ClosedPublic

Authored by IncludeGuardian on May 20 2023, 8:25 AM.

Details

Summary

Remove the unnecessary "llvm/IR/PatternMatch.h" include directive from
ComplexDeinterleavingPass.h and move it to the corresponding source
file.

Add missing includes that were transitively included by this header to 2
other source files.

This reduces the total number of preprocessing tokens across the LLVM
source files in lib from (roughly) 1,964,876,961 to 1,935,091,611 - a
reduction of ~1.52%. This should result in a small improvement in
compilation time.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptMay 20 2023, 8:25 AM
Herald added subscribers: pmatos, asb, foad and 8 others. · View Herald Transcript
IncludeGuardian requested review of this revision.May 20 2023, 8:25 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 20 2023, 8:25 AM
llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp
24

This review is a resubmission of https://reviews.llvm.org/D150514 as it broke the build since there was a more recent change to AMDGPUISelLowering.cpp that needs this include.

barannikov88 accepted this revision.May 20 2023, 9:16 AM
This revision is now accepted and ready to land.May 20 2023, 9:16 AM