Uh oh!
There was an error while loading. Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork 33.9k
gh-104469 : Convert _testcapi/vectorcall_limited.c to use AC#107857
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
bedevere-bot commented Aug 11, 2023
Most changes to Python require a NEWS entry. Please add it using the blurb_it web app or the blurb command-line tool. |
nahyeon-an commented Aug 11, 2023
corona10 left a comment
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.
lgtm!
ambv commented Aug 11, 2023
Closing and re-opening to retrigger CLA checks. Sorry for the noise. |
ghost commented Aug 11, 2023 • edited by ghost
Loading Uh oh!
There was an error while loading. Please reload this page.
edited by ghost
Uh oh!
There was an error while loading. Please reload this page.
bedevere-bot commented Aug 12, 2023
|
bedevere-bot commented Aug 12, 2023
|
bedevere-bot commented Aug 12, 2023
|
brettcannon commented Aug 14, 2023
@corona10 FYI this caused a few of the WebAssembly buildbots to fail, e.g., https://buildbot.python.org/all/#/builders/1046/builds/2731 . |
corona10 commented Aug 14, 2023
I will take a look, it can be easily fixed if the implicit function declarations is a real issue.
|
corona10 commented Aug 14, 2023 • edited
Loading Uh oh!
There was an error while loading. Please reload this page.
edited
Uh oh!
There was an error while loading. Please reload this page.
|
corona10 commented Aug 14, 2023
Okay, this is an issue between AC and limited C APIs. We should revert it first. Thank you @vstinner for investigation :) |
…se AC (pythongh-107857)" This reverts commit 2e27da1.
…se AC (pythongh-107857)" This reverts commit 2e27da1.
vstinner commented Aug 14, 2023
Right, see: |
erlend-aasland commented Aug 15, 2023
Sounds good, go ahead with any needed reverts. It is not important that the _test* extension modules use clinic :) |
…se AC … (pythongh-107951) Revert "pythongh-104469 : Convert _testcapi/vectorcall_limited.c to use AC (pythongh-107857)" This reverts commit 2e27da1.
vstinner commented Sep 21, 2023
New attempt since Argument Clinic now supports the limited C API: PR #109691. |
vstinner commented Sep 25, 2023
@nahyeon-an: I merged again your PR as 8c521f0 (PR gh-109691). Thanks for your contribution! |
Uh oh!
There was an error while loading. Please reload this page.