Uh oh!
There was an error while loading. Please reload this page.
gh-104469 : Convert _testcapi/vectorcall_limited.c to use AC - #107857
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
ambv
commented
Aug 11, 2023
Closing and re-opening to retrigger CLA checks. Sorry for the noise. |
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
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.