original ) (raw ) Issue27956
Created on 2016-09-05 19:02 by benjamin.peterson , last changed 2022-04-11 14:58 by admin . This issue is now closed .
Messages (4)
msg274420 - (view) Author: Benjamin Peterson (benjamin.peterson) *
Date: 2016-09-05 19:02
dict_traverse shows up a lot on statistical profiles of Python programs. This is not surprising since Python programs have lots of dicts. There isn't a huge amount of room for optimization here, but we can avoid some memory indirection. This patch saves a few percent of time through this function in my measurements.
msg274421 - (view) Author: Inada Naoki (methane) *
Date: 2016-09-05 19:11
lgtm. But http://bugs.python.org/issue27350 conflicts this.
msg274422 - (view) Author: Benjamin Peterson (benjamin.peterson) *
Date: 2016-09-05 19:13
Thank you for the review. On Mon, Sep 5, 2016, at 12:11, INADA Naoki wrote: > > INADA Naoki added the comment: > > lgtm. > But http://bugs.python.org/issue27350 conflicts this. > > ---------- > nosy: +methane > > _______________________________________ > Python tracker <report@bugs.python.org > > <http://bugs.python.org/issue27956 > > _______________________________________
msg274423 - (view) Author: Roundup Robot (python-dev)
Date: 2016-09-05 19:13
New changeset 027e421594b7 by Benjamin Peterson in branch 'default': remove memory indirections in dict_traverse (closes #27956 ) https://hg.python.org/cpython/rev/027e421594b7
History
Date
User
Action
Args
2022-04-11 14:58:35
admin
set
github: 72143
2016-09-05 19:13:25
python-dev
set
status: open -> closednosy: + python-dev messages: + resolution: fixedstage: resolved
2016-09-05 19:13:10
benjamin.peterson
set
messages: +
2016-09-05 19:11:48
methane
set
nosy: + methane messages: +
2016-09-05 19:02:13
benjamin.peterson
create
