Title: Do not build the freeze module twice in case of Windows PGO builds · Issue #130419 · python/cpython · GitHub
Open Graph Title: Do not build the freeze module twice in case of Windows PGO builds · Issue #130419 · python/cpython
X Title: Do not build the freeze module twice in case of Windows PGO builds · Issue #130419 · python/cpython
Description: Feature or enhancement Proposal: From #129907 (comment) Avoiding the rebuild of _freeze_module and the refreezing would be nice, but it's not a huge deal (a few seconds in a ~hour long release process). Don't mix it in with this PR. This...
Open Graph Description: Feature or enhancement Proposal: From #129907 (comment) Avoiding the rebuild of _freeze_module and the refreezing would be nice, but it's not a huge deal (a few seconds in a ~hour long release proc...
X Description: Feature or enhancement Proposal: From #129907 (comment) Avoiding the rebuild of _freeze_module and the refreezing would be nice, but it's not a huge deal (a few seconds in a ~hour long release ...
Opengraph URL: https://github.com/python/cpython/issues/130419
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Do not build the freeze module twice in case of Windows PGO builds","articleBody":"# Feature or enhancement\n\n### Proposal:\n\nFrom https://github.com/python/cpython/pull/129907#issuecomment-2648775431\n\u003e Avoiding the rebuild of `_freeze_module` and the refreezing would be nice, but it's not a huge deal (a few seconds in a ~hour long release process). Don't mix it in with this PR.\n\nThis is another low hanging fruit to make the Windows PGO build slightly faster (like https://github.com/python/cpython/issues/90281 did) by not building the _freeze_module again in the PGUpdate phase.\n\nThis will decrease both the MSVC and the clang-cl PGO build time (https://github.com/python/cpython/issues/130090).\n\nIn case of clang-cl, this saves me up to 40 seconds on my machine. Please see https://github.com/python/cpython/pull/131005 `Details pgupdate build times` for more numbers\n\n### Has this already been discussed elsewhere?\n\nThis is a minor feature, which does not need previous discussion elsewhere\n\n### Links to previous discussion of this feature:\n\n_No response_\n\n\u003c!-- gh-linked-prs --\u003e\n### Linked PRs\n* gh-130420\n\u003c!-- /gh-linked-prs --\u003e\n","author":{"url":"https://github.com/chris-eibl","@type":"Person","name":"chris-eibl"},"datePublished":"2025-02-21T19:01:04.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/130419/cpython/issues/130419"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:17dcd419-3f84-6555-2179-696b487d0bf0 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | AC18:2D4000:1CACE72:27FBD89:696A56D5 |
| html-safe-nonce | 43713f77625bb765351e4806c768d934a30feae146efc378b5a42553aae93da2 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJBQzE4OjJENDAwMDoxQ0FDRTcyOjI3RkJEODk6Njk2QTU2RDUiLCJ2aXNpdG9yX2lkIjoiNTc2MDkwODQ2Mjc0NDI5NTEyNSIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 003917a7ea93668318e34ff855a8749cbd984a1597db93fec34aeff35361e400 |
| hovercard-subject-tag | issue:2869796993 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/python/cpython/130419/issue_layout |
| twitter:image | https://opengraph.githubassets.com/2459fdd0d04919d01ecbc5c512f6c07df8f8b0b9c20c369a1b0771dc22bbfe2b/python/cpython/issues/130419 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/2459fdd0d04919d01ecbc5c512f6c07df8f8b0b9c20c369a1b0771dc22bbfe2b/python/cpython/issues/130419 |
| og:image:alt | Feature or enhancement Proposal: From #129907 (comment) Avoiding the rebuild of _freeze_module and the refreezing would be nice, but it's not a huge deal (a few seconds in a ~hour long release proc... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | chris-eibl |
| hostname | github.com |
| expected-hostname | github.com |
| None | 3f871c8e07f0ae1886fa8dac284166d28b09ad5bada6476fc10b674e489788ef |
| turbo-cache-control | no-preview |
| go-import | github.com/python/cpython git https://github.com/python/cpython.git |
| octolytics-dimension-user_id | 1525981 |
| octolytics-dimension-user_login | python |
| octolytics-dimension-repository_id | 81598961 |
| octolytics-dimension-repository_nwo | python/cpython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 81598961 |
| octolytics-dimension-repository_network_root_nwo | python/cpython |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 63c426b30d262aba269ef14c40e3c817b384cd61 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width