diff options
author | Dong-hee Na <donghee.na@python.org> | 2021-03-22 10:01:14 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-03-22 10:01:14 (GMT) |
commit | 86883d40e93acae980e52b90fddd7d042e439beb (patch) | |
tree | e5d189e9561d5c16f65d1f0c5bbe78fe646cfee7 /Python/bltinmodule.c | |
parent | 88d9983b561cd59e5f186d98227de0c1a022b498 (diff) | |
download | cpython-86883d40e93acae980e52b90fddd7d042e439beb.zip cpython-86883d40e93acae980e52b90fddd7d042e439beb.tar.gz cpython-86883d40e93acae980e52b90fddd7d042e439beb.tar.bz2 |
bpo-43575: Use PEP 590 vectorcall to speed up map() (GH-24955)
Diffstat (limited to 'Python/bltinmodule.c')
-rw-r--r-- | Python/bltinmodule.c | 45 |
1 files changed, 43 insertions, 2 deletions
diff --git a/Python/bltinmodule.c b/Python/bltinmodule.c index 4683103..9f83c03 100644 --- a/Python/bltinmodule.c +++ b/Python/bltinmodule.c @@ -1264,8 +1264,48 @@ map_new(PyTypeObject *type, PyObject *args, PyObject *kwds) } lz->iters = iters; func = PyTuple_GET_ITEM(args, 0); - Py_INCREF(func); - lz->func = func; + lz->func = Py_NewRef(func); + + return (PyObject *)lz; +} + +static PyObject * +map_vectorcall(PyObject *type, PyObject * const*args, + size_t nargsf, PyObject *kwnames) +{ + PyTypeObject *tp = (PyTypeObject *)type; + if (tp == &PyMap_Type && !_PyArg_NoKwnames("map", kwnames)) { + return NULL; + } + + Py_ssize_t nargs = PyVectorcall_NARGS(nargsf); + if (nargs < 2) { + PyErr_SetString(PyExc_TypeError, + "map() must have at least two arguments."); + return NULL; + } + + PyObject *iters = PyTuple_New(nargs-1); + if (iters == NULL) { + return NULL; + } + + for (int i=1; i<nargs; i++) { + PyObject *it = PyObject_GetIter(args[i]); + if (it == NULL) { + Py_DECREF(iters); + return NULL; + } + PyTuple_SET_ITEM(iters, i-1, it); + } + + mapobject *lz = (mapobject *)tp->tp_alloc(tp, 0); + if (lz == NULL) { + Py_DECREF(iters); + return NULL; + } + lz->iters = iters; + lz->func = Py_NewRef(args[0]); return (PyObject *)lz; } @@ -1403,6 +1443,7 @@ PyTypeObject PyMap_Type = { PyType_GenericAlloc, /* tp_alloc */ map_new, /* tp_new */ PyObject_GC_Del, /* tp_free */ + .tp_vectorcall = (vectorcallfunc)map_vectorcall }; |