Skip to content

gh-89013: Improve the performance of methodcaller (lazy version) #107201

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.

Sign up for GitHub

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

Merged
corona10 merged 21 commits into from
Aug 1, 2023
Merged

gh-89013: Improve the performance of methodcaller (lazy version) #107201

Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
12a80bd
bpo-44850: Speedup methodcaller via vectorcall.
anntzer Aug 16, 2021
3d19683
Merge branch 'main' into fastmethodcaller
eendebakpt Jul 18, 2023
035456c
Merge branch 'main' into fastmethodcaller
eendebakpt Jul 21, 2023
8be0aed
optimize methodcaller construction
eendebakpt Jul 21, 2023
83c76bb
fix bug in methodcaller_reduce
eendebakpt Jul 21, 2023
09e3ee3
restore methodcaller_call
eendebakpt Jul 21, 2023
5d715c5
avoid looping over the dict twice
eendebakpt Jul 23, 2023
a6c465b
Merge branch 'main' into fastmethodcaller
eendebakpt Jul 23, 2023
486c498
lazy allocation of vectorcall arguments
eendebakpt Jul 24, 2023
f174256
fix double free
eendebakpt Jul 24, 2023
d70a8ad
Merge branch 'main' into fastmethodcaller_lazy_vectorcall
eendebakpt Jul 24, 2023
1ceea51
Apply suggestions from code review
eendebakpt Jul 30, 2023
608828a
Update Modules/_operator.c
eendebakpt Jul 30, 2023
ad7d6bd
inline methodcaller_clear
eendebakpt Jul 30, 2023
3c3d73f
Merge branch 'fastmethodcaller_lazy_vectorcall' of githubeendebakpt:e…
eendebakpt Jul 30, 2023
9335d6d
fix vectorcall initialization check
eendebakpt Jul 30, 2023
0c050fc
review comments
eendebakpt Jul 31, 2023
626169f
Merge branch 'main' into fastmethodcaller_lazy_vectorcall
eendebakpt Jul 31, 2023
0634f66
fix merge conflicts
eendebakpt Jul 31, 2023
5c56b64
Merge branch 'main' into fastmethodcaller_lazy_vectorcall
eendebakpt Jul 31, 2023
812856b
fix bug in clear
eendebakpt Jul 31, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
review comments
  • Loading branch information
@eendebakpt
eendebakpt committed Jul 31, 2023
commit 0c050fc20234afeb939383edf886da016083df69
11 changes: 6 additions & 5 deletions Modules/_operator.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1556,7 +1556,7 @@ typedef struct {
vectorcallfunc vectorcall;
} methodcallerobject;

static void* _methodcaller_initialize_vectorcall(methodcallerobject* mc)
static int _methodcaller_initialize_vectorcall(methodcallerobject* mc)
{
PyObject* args = mc->xargs;
PyObject* kwds = mc->kwds;
Expand All @@ -1567,7 +1567,8 @@ static void* _methodcaller_initialize_vectorcall(methodcallerobject* mc)
nargs + (kwds ? PyDict_Size(kwds) : 0),
sizeof(PyObject*));
if (!mc->vectorcall_args) {
return PyErr_NoMemory();
PyErr_NoMemory();
return -1;
}
/* The first item of vectorcall_args will be filled with obj later */
if (nargs > 1) {
Expand All @@ -1579,7 +1580,7 @@ static void* _methodcaller_initialize_vectorcall(methodcallerobject* mc)

mc->vectorcall_kwnames = PyTuple_New(nkwds);
if (!mc->vectorcall_kwnames) {
return NULL;
return -1;
}
Py_ssize_t i = 0, ppos = 0;
PyObject* key, * value;
Expand All @@ -1592,7 +1593,7 @@ static void* _methodcaller_initialize_vectorcall(methodcallerobject* mc)
else {
mc->vectorcall_kwnames = NULL;
}
return (void *)1;
return 1;
}


Expand All @@ -1605,7 +1606,7 @@ methodcaller_vectorcall(
return NULL;
}
if (mc->vectorcall_args == NULL) {
if (_methodcaller_initialize_vectorcall(mc) == NULL) {
if (_methodcaller_initialize_vectorcall(mc) < 0) {
return NULL;
}
}
Expand Down