classification
Title: Replace /Ox with /O2
Type: enhancement Stage: patch review
Components: Distutils Versions: Python 3.9
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: Nosy List: Kojoley, dstufft, eric.araujo, steve.dower
Priority: normal Keywords: patch

Created on 2019-08-08 17:14 by Kojoley, last changed 2019-09-09 15:50 by steve.dower.

Pull Requests
URL Status Linked Edit
PR 15179 open Kojoley, 2019-08-08 17:14
Messages (2)
msg349243 - (view) Author: Nikita Kniazev (Kojoley) * Date: 2019-08-08 17:14
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
msg351507 - (view) Author: Steve Dower (steve.dower) * (Python committer) Date: 2019-09-09 15:50
Given the potential compatibility impact (small chance of third-party extensions relying on function aliasing), I'd like to see an update to Docs/whatsnew/3.9.rst as well.

Also the suggested change to the NEWS entry is good. I don't think this one should pass completely silently.
History
Date User Action Args
2019-09-09 15:50:24steve.dowersetnosy: + steve.dower

messages: + msg351507
versions: + Python 3.9
2019-08-08 17:14:44Kojoleysetkeywords: + patch
stage: patch review
pull_requests: + pull_request14910
2019-08-08 17:14:02Kojoleycreate