◐ Shell
reader mode source ↗
Skip to content

GH-130090: build time analysis#131005

Closed
chris-eibl wants to merge 22 commits into
python:mainfrom
chris-eibl:analyse_build_times
Closed

GH-130090: build time analysis#131005
chris-eibl wants to merge 22 commits into
python:mainfrom
chris-eibl:analyse_build_times

Conversation

@chris-eibl

@chris-eibl chris-eibl commented Mar 9, 2025

Copy link
Copy Markdown
Member

This is how I generated the tables in #130090.

What to do with it?

Don't know, whether we want anything of that branch. Most probably just close.

chris-eibl and others added 21 commits February 9, 2025 16:30
for _freeze_module in case of clang-cl to speed up the build
Speeds up both MSVC and clang-cl builds.

Should most probably done in a separate PR and issue, though.
I've previously gotten compile errors from clang, because the needed
intrinsics were not available without that option.

Cannot reproduce anymore. Most probably, because I've upgraded to
Visual Studio 17.13.0 Preview 5.0, which now ships with clang 19.1.1
instead of 18.1.8 and they've done that for compatibility with MSVC?

Anyway, let's keep the PR small :)
This reverts commit 26fb51f.

Shall be done in a separate PR.
This better matches the behaviour of build.bat in case of MSVC PGO builds.
and make it a target with inputs and outputs
because the name is too MSVC specific
Co-authored-by: Steve Dower <steve.dower@microsoft.com>
@chris-eibl chris-eibl changed the title use -flto=thin for clang-cl on Windows Mar 10, 2025
@chris-eibl

Copy link
Copy Markdown
Member Author

Closing, since this served its purpose. Build time analysis is done, in case someone else ever wants to do something like that:
PCbuild/analyse_build_times.py and BeginTimeStamp/EndTimeStamp is the only stuff needed ...

@chris-eibl chris-eibl closed this Mar 22, 2025
@chris-eibl chris-eibl deleted the analyse_build_times branch March 22, 2025 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant