-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
bpo-42692: fix __builtin_available check on older compilers #23873
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
A compiler that doesn't define __has_builtin will error out when it is used on the same line as the check for it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch.
@jmroot: Status check is done, and it's a success ✅ . |
1 similar comment
@jmroot: Status check is done, and it's a success ✅ . |
Sorry, I can't merge this PR. Reason: |
Sorry, I can't merge this PR. Reason: |
@jmroot: Status check is done, and it's a success ✅ . |
Thanks @jmroot for the PR 🌮🎉.. I'm working now to backport this PR to: 3.9. |
…-23873) A compiler that doesn't define `__has_builtin` will error out when it is used on the same line as the check for it. Automerge-Triggered-By: GH:ronaldoussoren (cherry picked from commit df21f50) Co-authored-by: Joshua Root <[email protected]>
GH-24090 is a backport of this pull request to the 3.9 branch. |
GH-24090) A compiler that doesn't define `__has_builtin` will error out when it is used on the same line as the check for it. Automerge-Triggered-By: GH:ronaldoussoren (cherry picked from commit df21f50) Co-authored-by: Joshua Root <[email protected]>
…-23873) A compiler that doesn't define `__has_builtin` will error out when it is used on the same line as the check for it. Automerge-Triggered-By: GH:ronaldoussoren
A compiler that doesn't define
__has_builtin
will error out when it isused on the same line as the check for it.
https://bugs.python.org/issue42692
Automerge-Triggered-By: GH:ronaldoussoren