Uh oh!
There was an error while loading. Please reload this page.
GH-131288: use _AddressOfReturnAddress in _Py_get_machine_stack_pointer to silence MSVC warning in pycore_ceval.h for release builds - #131289
Conversation
| #ifdef _MSC_VER | ||
| #pragma warning(push) | ||
| #pragma warning(disable:4172) | ||
| #endif |
There was a problem hiding this comment.
Use _AddressOfReturnAddress and avoid the UB on Windows?
https://learn.microsoft.com/en-us/cpp/intrinsics/addressofreturnaddress?view=msvc-170
There was a problem hiding this comment.
Great - even better!
See https://godbolt.org/z/7c9Gq9jzM, that should work out?
There was a problem hiding this comment.
Yeah, it works on Windows arm64 too.
colesbury
commented
Mar 15, 2025
GitHub actions was having some trouble, so I merged main into the PR to give it a kick |
colesbury
commented
Mar 15, 2025
!buildbot Windows |
bedevere-bot
commented
Mar 15, 2025
🤖 New build scheduled with the buildbot fleet by @colesbury for commit 66f5bc5 🤖 Results will be shown at: https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F131289%2Fmerge The command will test the builders whose names match following regular expression: The builders matched are:
|
chris-eibl
commented
Mar 15, 2025
Oh, my beloved See #114911. |
colesbury
commented
Mar 17, 2025
@markshannon - would you like to review this? |
Shall we get this into the last alpha? Or before beta 1? Or no need at all? The AMD64 Windows PGO NoGIL failure is definitely unrelated, if this is blocking ... |
colesbury
commented
Apr 4, 2025
Thanks for the reminder |
For details please see #131288.
I think this is a skip news.