Skip to content

Conversation

@colesbury
Copy link
Contributor

@colesbury colesbury commented Mar 18, 2024

@colesbury
Copy link
Contributor Author

Tested with:

CFLAGS="-D_Py_USE_GCC_BUILTIN_ATOMICS=0" ./configure --with-pydebug --disable-gil

Once @vstinner lands #116954, we may want to extend the test to also try compiling extensions with -D_Py_USE_GCC_BUILTIN_ATOMICS=0

Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Oh wow, surprising that we missed that. But I remeber that the review was complicated and I modified the code, so it can be my fault :-)

@colesbury
Copy link
Contributor Author

It wasn't in the original pyatomic_std.h. I think it happened later when resolving a merge conflict.

@colesbury colesbury merged commit 165cb45 into python:main Mar 18, 2024
@colesbury colesbury deleted the gh-116941-pyatomic-std branch March 18, 2024 21:05
vstinner pushed a commit to vstinner/cpython that referenced this pull request Mar 20, 2024
adorilson pushed a commit to adorilson/cpython that referenced this pull request Mar 25, 2024
diegorusso pushed a commit to diegorusso/cpython that referenced this pull request Apr 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants