-
-
Notifications
You must be signed in to change notification settings - Fork 32k
gh-93274: Expose receiving vectorcall in the Limited API #95717
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
This looks great. Leaving |
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.
Looks good, with minor nits!
BTW, in Modules/_testcapi/vectorcall_limited.c
, everything but _PyTestCapi_Init_VectorcallLimited
method can be made static
. Not that it matters much for a test module, though.
Objects/call.c
Outdated
@@ -1047,3 +1047,11 @@ _PyStack_UnpackDict_Free(PyObject *const *stack, Py_ssize_t nargs, | |||
PyMem_Free((PyObject **)stack - 1); | |||
Py_DECREF(kwnames); | |||
} | |||
|
|||
// Export for the stable ABI | |||
#undef PyVectorcall_NARGS(n) |
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.
#undef PyVectorcall_NARGS(n) | |
#undef PyVectorcall_NARGS |
|
pythonGH-95717)" This reverts commit 656dad7.
If anyone needs the TraceRefs buildbot green, please merge the revert. I'm working on a fix, so maybe a revert won't be necessary. |
https://buildbot.python.org/all/#builders/484/builds/1920 failed to build:
IMO it's ok if it's broken for a few days, but it would be nice to fix it before next Python (3.12) release. |
This'll hopefully fix it: #95796 |
Uh oh!
There was an error while loading. Please reload this page.