Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve compilation time with include-what-you-use (editor folder) #57734

Closed
wants to merge 1 commit into from

Conversation

Geometror
Copy link
Member

@Geometror Geometror commented Feb 7, 2022

Similar to #57641, but in this PR the includes are generated by include-what-you-use (there was still some manual work required). The changes are restricted to the editor folder since this cleanup should be done bit for bit.
[Alternative to #57641 which is purely manual]

EDIT: Part of a series of PRs to reduce the compilation time with include-what-you-use together with some manual fixes due to the complexity of the codebase. [see #57760, #57759]

Although the full potential will only be developed when iwyu is run over a larger part of the codebase, there is already a significant improvement:

1 file changed (filesystem_dock.h)
0:43 min -> 0:33 min
4 files changed (editor_file_dialog.h/file_dialog.h/dependency_editor.cpp/filesystem_dock.h)
4:07 min -> 3:23 min

@Geometror Geometror requested review from a team as code owners February 7, 2022 01:33
@KoBeWi KoBeWi added this to the 4.0 milestone Feb 7, 2022
@Geometror Geometror changed the title Improve compilation times with include-what-you-use (editor folder) Improve compilation time with include-what-you-use (editor folder) Feb 7, 2022
@YuriSizov YuriSizov modified the milestones: 4.0, 4.x Sep 8, 2022
@YuriSizov
Copy link
Contributor

Closing, see #57760 (comment).

@YuriSizov YuriSizov closed this Apr 11, 2023
@YuriSizov YuriSizov removed this from the 4.x milestone Dec 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants