Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

bpo-43166: Disable ceval.c optimizations for Windows debug builds again#32023

Merged
zooba merged 10 commits into
python:mainpython/cpython:mainfrom
neonene:bpo-43166Copy head branch name to clipboard
Mar 23, 2022
Merged

bpo-43166: Disable ceval.c optimizations for Windows debug builds again#32023
zooba merged 10 commits into
python:mainpython/cpython:mainfrom
neonene:bpo-43166Copy head branch name to clipboard

Conversation

@neonene

@neonene neonene commented Mar 21, 2022

Copy link
Copy Markdown
Contributor

Debug builds' optimization could be reconsidered as 3.11a6+ has reduced the stack usage. Disabling on MSVC should fix the build performance regression and make ceval.c debuggable.

test_xml_etree, which is the top stack consumer among 17 tests mentioned in bpo-43271, requires the following bytes:

Optimization x64 x86
Enabled (/Og /Ot) 1,100,000 500,000
Disabled 3,900,000 2,300,000

They include 200,000 bytes usage with /Ob1 (OnlyExplicitInline) option introduced by #10094 for running performance.

https://bugs.python.org/issue43166

@neonene

neonene commented Mar 21, 2022

Copy link
Copy Markdown
Contributor Author

Small improvement on GitHub test machines.

Build time x64 x86
before (e63894b) 3:15.28 3:10.90
This (1ebe5f4 ) 2:54.56 2:29.58

Comment thread Include/pyport.h Outdated
Comment thread PCbuild/_freeze_module.vcxproj Outdated
@zooba zooba merged commit cd05d0a into python:main Mar 23, 2022
@neonene

neonene commented Mar 23, 2022

Copy link
Copy Markdown
Contributor Author

Thanks!

@neonene neonene deleted the bpo-43166 branch March 23, 2022 01:04
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.