Skip to content

Disable Whole Program Optimization on MSVC#915

Open
azeey wants to merge 1 commit intomainfrom
ci_matching_branch/gz_msvc_wpo
Open

Disable Whole Program Optimization on MSVC#915
azeey wants to merge 1 commit intomainfrom
ci_matching_branch/gz_msvc_wpo

Conversation

@azeey
Copy link
Contributor

@azeey azeey commented Mar 23, 2026

🦟 Bug fix

Summary

This leverages the new GZ_MSVC_WPO option in gz-cmake to globally disable Whole Program Optimization (which removes /GL and /LTCG) to significantly speed up Windows build times.

Checklist

  • Signed all commits for DCO
  • Added a screen capture or video to the PR description that demonstrates the fix (as needed)
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • Updated Bazel files (if adding new files). Created an issue otherwise.
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers
  • Was GenAI used to generate this PR? If so, make sure to add "Generated-by" to your commits. (See this policy for more info.)

Generated-By: Gemini 3.1 Pro

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by and Generated-by messages.

Backports: If this is a backport, please use Rebase and Merge instead.

This leverages the new GZ_MSVC_WPO option in gz-cmake to globally disable Whole Program Optimization (which removes /GL and /LTCG) to significantly speed up Windows build times.

Generated-By: Gemini 3.1 Pro
Signed-off-by: Addisu Z. Taddese <addisuzt@intrinsic.ai>
@github-project-automation github-project-automation bot moved this from Inbox to In review in Core development Mar 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

2 participants