◐ Shell
reader mode source ↗
Skip to content

bpo-35059 : Add /Ob1 flag when building pythoncore in debug mode#10094

Merged
vstinner merged 1 commit into
python:masterfrom
vstinner:msvc_Ob1
Oct 26, 2018
Merged

bpo-35059 : Add /Ob1 flag when building pythoncore in debug mode#10094
vstinner merged 1 commit into
python:masterfrom
vstinner:msvc_Ob1

Conversation

@vstinner

@vstinner vstinner commented Oct 25, 2018 •
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

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

@vstinner vstinner requested review from zooba and zware October 25, 2018 15:18
@vstinner vstinner requested a review from a team as a code owner October 25, 2018 15:18
@vstinner

vstinner commented Oct 25, 2018

Copy link
Copy Markdown
Member Author

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:

       -Og Optimize debugging experience.  -Og enables optimizations that do
           not interfere with debugging. It should be the optimization level
           of choice for the standard edit-compile-debug cycle, offering a
           reasonable level of optimization while maintaining fast compilation
           and a good debugging experience.

@zware

zware commented Oct 25, 2018

Copy link
Copy Markdown
Member

You can instead add this to PCbuild/pyproject.props in the existing <ClCompile Condition="$(Configuration) == 'Debug'"> section that disables optimization globally in Debug configuration.

@vstinner vstinner mentioned this pull request Oct 25, 2018
Merged
@vstinner

vstinner commented Oct 25, 2018

Copy link
Copy Markdown
Member Author

You can instead add this to PCbuild/pyproject.props in the existing section that disables optimization globally in Debug configuration.

Oh thanks, I will try that! I was looking for something similar, but I failed to find it in the Visual Studio UI.

@vstinner

vstinner commented Oct 26, 2018

Copy link
Copy Markdown
Member Author

You can instead add this to PCbuild/pyproject.props in the existing section that disables optimization globally in Debug configuration.

Oh, fantastic! I tried and it works as expected! It's exactly what I was looking for ;-) I didn't know pyproject.props.

@vstinner
bpo-35059, PCbuild: Expand inline funcs in Debug …
248cfef 
Visual Studio solution: Set InlineFunctionExpansion to
OnlyExplicitInline ("/Ob1" option) on all projects (in
pyproject.props) in Debug mode on Win32 and x64 platforms to expand
functions marked as inline.

This change should make Python compiled in Debug mode a little bit
faster on Windows. On Unix, GCC uses -Og optimization level for
./configure --with-pydebug.
@vstinner vstinner merged commit a05bef4 into python:master Oct 26, 2018
@vstinner vstinner deleted the msvc_Ob1 branch October 26, 2018 13:10
@bedevere-bot

bedevere-bot commented Oct 26, 2018

Copy link
Copy Markdown

Hi! The buildbot AMD64 Windows7 SP1 3.x has failed when building commit a05bef4.

You can take a look here:

https://buildbot.python.org/all/#builders/40/builds/1057

@vstinner

vstinner commented Oct 26, 2018

Copy link
Copy Markdown
Member Author

Hi! The buildbot AMD64 Windows7 SP1 3.x has failed when building commit a05bef4.

Oh. I broke this buildbot for real :-( The compilation of the _decimal module fails:

            Creating library C:\buildbot.python.org\3.x.kloth-win64\build\PCbuild\amd64\_decimal_d.lib and object C:\buildbot.python.org\3.x.kloth-win64\build\PCbuild\amd64\_decimal_d.exp
    19>ClCompile:
         xmltok.c
    16>ClCompile:
         prep_cif.c
    21>ClCompile:
         multiprocessing.c
    19>ClCompile:
         Generating Code...
    17>_decimal.obj : error LNK2019: unresolved external symbol mpd_del referenced in function PyDec_AsTuple [C:\buildbot.python.org\3.x.kloth-win64\build\PCbuild\_decimal.vcxproj]
    17>io.obj : error LNK2001: unresolved external symbol mpd_del [C:\buildbot.python.org\3.x.kloth-win64\build\PCbuild\_decimal.vcxproj]
    17>basearith.obj : error LNK2019: unresolved external symbol mpd_uint_zero referenced in function _mpd_baseshiftl [C:\buildbot.python.org\3.x.kloth-win64\build\PCbuild\_decimal.vcxproj]
    17>io.obj : error LNK2019: unresolved external symbol mpd_qresize referenced in function mpd_qset_string [C:\buildbot.python.org\3.x.kloth-win64\build\PCbuild\_decimal.vcxproj]
    17>C:\buildbot.python.org\3.x.kloth-win64\build\PCbuild\amd64\_decimal_d.pyd : fatal error LNK1120: 3 unresolved externals [C:\buildbot.python.org\3.x.kloth-win64\build\PCbuild\_decimal.vcxproj]
(...)
       "C:\buildbot.python.org\3.x.kloth-win64\build\PCbuild\_decimal.vcxproj" (Build target) (17) ->
       (Link target) -> 
         _decimal.obj : error LNK2019: unresolved external symbol mpd_del referenced in function PyDec_AsTuple [C:\buildbot.python.org\3.x.kloth-win64\build\PCbuild\_decimal.vcxproj]
         io.obj : error LNK2001: unresolved external symbol mpd_del [C:\buildbot.python.org\3.x.kloth-win64\build\PCbuild\_decimal.vcxproj]
         basearith.obj : error LNK2019: unresolved external symbol mpd_uint_zero referenced in function _mpd_baseshiftl [C:\buildbot.python.org\3.x.kloth-win64\build\PCbuild\_decimal.vcxproj]
         io.obj : error LNK2019: unresolved external symbol mpd_qresize referenced in function mpd_qset_string [C:\buildbot.python.org\3.x.kloth-win64\build\PCbuild\_decimal.vcxproj]
         C:\buildbot.python.org\3.x.kloth-win64\build\PCbuild\amd64\_decimal_d.pyd : fatal error LNK1120: 3 unresolved externals [C:\buildbot.python.org\3.x.kloth-win64\build\PCbuild\_decimal.vcxproj]

@bedevere-bot

bedevere-bot commented Oct 26, 2018

Copy link
Copy Markdown

Hi! The buildbot AMD64 Windows8 3.x has failed when building commit a05bef4.

You can take a look here:

https://buildbot.python.org/all/#builders/32/builds/1639

@bedevere-bot

bedevere-bot commented Oct 26, 2018

Copy link
Copy Markdown

Hi! The buildbot AMD64 Windows10 3.x has failed when building commit a05bef4.

You can take a look here:

https://buildbot.python.org/all/#builders/3/builds/1566

vstinner added a commit that referenced this pull request Oct 26, 2018
@vstinner
Revert "bpo-35059, PCbuild: Expand inline funcs in Debug (GH-10094)" …
59ff430 
This reverts commit a05bef4.
This was referenced Oct 26, 2018
Closed
Merged
@vstinner

vstinner commented Oct 26, 2018

Copy link
Copy Markdown
Member Author

It seems like my PR #10128 fixed the linker error.

@neonene neonene mentioned this pull request Mar 21, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zooba zooba Awaiting requested review from zooba

@zware zware Awaiting requested review from zware

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@vstinner @zware @bedevere-bot @the-knights-who-say-ni