bpo-37794: MSVCCompiler: Replace /Ox with /O2 #15179
Conversation
The /O2 is a superset of /Ox with additional /GF and /Gy switches which enables strings and functions deduplication and almost always are favorable optimizations without downsides. https://docs.microsoft.com/en-us/cpp/build/reference/ox-full-optimization
Kojoley
force-pushed
the
msvc-replace-ox-with-o2
branch
from
b7a8c5b to
47bfffd
Compare
There was a problem hiding this comment.
Sounds good.
Do we also need to change this line?
cpython/Tools/freeze/winmakemakefile.py
Line 68 in 5affd23
| @@ -0,0 +1 @@ | |||
| :class:`distutils.MSVCCompiler` now uses ``/O2`` switch instead of ``/Ox``. No newline at end of file | |||
There was a problem hiding this comment.
| :class:`distutils.MSVCCompiler` now uses ``/O2`` switch instead of ``/Ox``. | |
| :class:`distutils.MSVCCompiler` now uses the ``/O2`` optimization instead of ``/Ox``. |
I know nothing about that code. It does not belong to |
|
It's been a year, any progress on this? @Kojoley, you've got a feedback on bug tracker. Could you please finish with this? |
|
No, I am sorry, there is too much time consuming bureaucracy for a simple thing. |
|
Closing this PR following the closure of its b.p.o. issue (https://bugs.python.org/issue37794) |
The /O2 is a superset of /Ox with additional /GF and /Gy switches which
enables strings and functions deduplication and almost always are
favorable optimizations without downsides.
https://docs.microsoft.com/en-us/cpp/build/reference/ox-full-optimization
https://bugs.python.org/issue37794