FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

bpo-35059 : Add /Ob1 flag when building pythoncore in debug mode by vstinner · Pull Request #10094 · python/cpython · GitHub

/ cpython Public

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
ConversationCommits (1)ChecksFiles changed
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 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 requested review from zooba and zware October 25, 2018 15:18
vstinner requested a review from a team as a code owner October 25, 2018 15:18

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 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 mentioned this pull request Oct 25, 2018
Merged

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 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.

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 merged commit a05bef4 into python:master Oct 26, 2018
vstinner deleted the msvc_Ob1 branch October 26, 2018 13:10

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 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 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 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
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 commented Oct 26, 2018

Copy link
Copy Markdown
Member Author

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

neonene mentioned this pull request Mar 21, 2022
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

zooba Awaiting requested review from zooba
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

Back | FazBrowse Home | New Git URL