This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author vstinner
Recipients python-dev, serhiy.storchaka, vstinner, xiang.zhang
Date 2017-01-11.00:20:39
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1484094039.97.0.430484628044.issue28870@psf.upfronthosting.co.za>
In-reply-to
Content
I pushed 3 changes:

* rev b9404639a18c: Issue #29233: call_method() now uses _PyObject_FastCall()
* rev 8481c379e2da: Issue #29227: inline call_function()
* rev 6478e6d0476f: Issue #29234: disable _PyStack_AsTuple() inlining


Before (rev a30cdf366c02):

test_python_call: 7175 calls before crash, stack: 1168 bytes/call
test_python_getitem: 6235 calls before crash, stack: 1344 bytes/call
test_python_iterator: 5344 calls before crash, stack: 1568 bytes/call

=> total: 18754 calls, 4080 bytes


With these 3 changes (rev 6478e6d0476f):

test_python_call: 8587 calls before crash, stack: 976 bytes/call
test_python_getitem: 9189 calls before crash, stack: 912 bytes/call
test_python_iterator: 7936 calls before crash, stack: 1056 bytes/call

=> total: 25712 calls, 2944 bytes


The default branch is now as good as Python 3.4, in term of stack consumption, and Python 3.4 was the Python version which used the least stack memory according to my tests.

I didn't touch _PY_FASTCALL_SMALL_STACK value, it's still 5 arguments (40 bytes). So my changes should not impact performances.
History
Date User Action Args
2017-01-11 00:20:40vstinnersetrecipients: + vstinner, python-dev, serhiy.storchaka, xiang.zhang
2017-01-11 00:20:39vstinnersetmessageid: <1484094039.97.0.430484628044.issue28870@psf.upfronthosting.co.za>
2017-01-11 00:20:39vstinnerlinkissue28870 messages
2017-01-11 00:20:39vstinnercreate