[issue43166] Unused letters in Windows-specific pragma optimize

[Steve Dower]

Steve Dower  added the comment:


New changeset cd05d0a423d97be69f9de4650f68f89e99ad68d1 by neonene in branch 
'main':
bpo-43166: Disable ceval.c optimizations for Windows debug builds (GH-32023)
https://github.com/python/cpython/commit/cd05d0a423d97be69f9de4650f68f89e99ad68d1


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43166] Unused letters in Windows-specific pragma optimize

[neonene]

Change by neonene :


--
nosy: +neonene
nosy_count: 7.0 -> 8.0
pull_requests: +30111
pull_request: https://github.com/python/cpython/pull/32023

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43166] Unused letters in Windows-specific pragma optimize

[David Bolen]

Change by David Bolen :


--
nosy: +db3l

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43166] Unused letters in Windows-specific pragma optimize

[STINNER Victor]

STINNER Victor  added the comment:

Since this change, AMD64 Windows10 3.x buildbot started to crash with Windows 
fatal exception: stack overflow -> see bpo-43271.

--
nosy: +vstinner

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43166] Unused letters in Windows-specific pragma optimize

[Steve Dower]

Steve Dower  added the comment:


New changeset b74396c3167cc780f01309148db02709bc37b432 by Steve Dower in branch 
'master':
bpo-43166: Disable ceval.c optimisations for Windows debug builds (GH-24485)
https://github.com/python/cpython/commit/b74396c3167cc780f01309148db02709bc37b432


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43166] Unused letters in Windows-specific pragma optimize

[Steve Dower]

Change by Steve Dower :


--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43166] Unused letters in Windows-specific pragma optimize

[Steve Dower]

Change by Steve Dower :


--
keywords: +patch
pull_requests: +23286
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/24485

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43166] Unused letters in Windows-specific pragma optimize

[Zachary Ware]

Change by Zachary Ware :


--
assignee:  -> steve.dower
components: +Windows
nosy: +paul.moore, steve.dower, tim.golden, zach.ware

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43166] Unused letters in Windows-specific pragma optimize

[Guido van Rossum]

New submission from Guido van Rossum :

A coworker happened to look at our use of #pragma optimize() for Windows (VS 
2017) and noticed:

unless there’s something I’m missing, the ‘a’ and ‘w’ portions of the string 
being passed to the optimize pragma are not doing anything; I’m pretty sure 
they’ve been useless for 10+ years. The ‘g’ means turn on global optimizations 
and the ‘t’ means optimize for “time” (i.e. speed). See the documentation at 
https://docs.microsoft.com/en-us/cpp/preprocessor/optimize?view=msvc-160

--
messages: 386646
nosy: gvanrossum
priority: normal
severity: normal
status: open
title: Unused letters in Windows-specific pragma optimize
versions: Python 3.10

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com