Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.
Sign upbpo-35059 : Add /Ob1 flag when building pythoncore in debug mode #10094
Conversation
vstinner
requested review from
zooba and
zware
Oct 25, 2018
vstinner
requested a review
from python/windows-team
as a
code owner
Oct 25, 2018
the-knights-who-say-ni
added
the
CLA signed
label
Oct 25, 2018
bedevere-bot
added
the
awaiting merge
label
Oct 25, 2018
This comment has been minimized.
This comment has been minimized.
|
This change matters when you consider the PR #10079 which converts Py_INCREF() macro to a static inline function. I'm not sure if it's ok to only modify pythoncore, or if the 38 other projects should be modified the same way? On Unix, Python is now compiled with -Og:
|
This comment has been minimized.
This comment has been minimized.
|
You can instead add this to PCbuild/pyproject.props in the existing |
This comment has been minimized.
This comment has been minimized.
Oh thanks, I will try that! I was looking for something similar, but I failed to find it in the Visual Studio UI. |
vstinner
force-pushed the
vstinner:msvc_Ob1
branch
from
ed3d60e
to
3fb77ff
Oct 26, 2018
This comment has been minimized.
This comment has been minimized.
Oh, fantastic! I tried and it works as expected! It's exactly what I was looking for ;-) I didn't know pyproject.props. |
vstinner
force-pushed the
vstinner:msvc_Ob1
branch
from
3fb77ff
to
248cfef
Oct 26, 2018
vstinner
merged commit a05bef4
into
python:master
Oct 26, 2018
5 checks passed
bedevere-bot
removed
the
awaiting merge
label
Oct 26, 2018
vstinner
deleted the
vstinner:msvc_Ob1
branch
Oct 26, 2018
This comment has been minimized.
This comment has been minimized.
bedevere-bot
commented
Oct 26, 2018
|
Hi! The buildbot AMD64 Windows7 SP1 3.x has failed when building commit a05bef4. You can take a look here: |
This comment has been minimized.
This comment has been minimized.
Oh. I broke this buildbot for real :-( The compilation of the _decimal module fails:
|
This comment has been minimized.
This comment has been minimized.
bedevere-bot
commented
Oct 26, 2018
|
Hi! The buildbot AMD64 Windows8 3.x has failed when building commit a05bef4. You can take a look here: |
This comment has been minimized.
This comment has been minimized.
bedevere-bot
commented
Oct 26, 2018
|
Hi! The buildbot AMD64 Windows10 3.x has failed when building commit a05bef4. You can take a look here: |
This comment has been minimized.
This comment has been minimized.
|
It seems like my PR #10128 fixed the linker error. |
vstinner commentedOct 25, 2018
•
edited by bedevere-bot
Visual Studio solution: the "pythoncore" project is now compiled with
the /Ob1 flag in Debug mode to expand functions marked as inline.
https://bugs.python.org/issue35059