[Patches] [ python-Patches-1337051 ] remove 4 ints from PyFrameObject
SourceForge.net
noreply at sourceforge.net
Tue May 23 20:59:58 CEST 2006
Patches item #1337051, was opened at 2005-10-25 02:18
Message generated for change (Comment added) made by tim_one
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1337051&group_id=5470
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core (C code)
Group: Python 2.5
>Status: Closed
>Resolution: Accepted
Priority: 5
Submitted By: Neal Norwitz (nnorwitz)
Assigned to: Nobody/Anonymous (nobody)
Summary: remove 4 ints from PyFrameObject
Initial Comment:
Decreases the size of each frame object by 32 bytes.
The 4 ints are already in the PyCodeObject. Well, 2
are in there directly (co_nlocals and co_stacksize).
The other 2 are the tuple lengths of co_cellvars and
co_freevars.
I ran pybench before and after the patch. With the
patch, the interpreter was .002 seconds slower, ie,
noise. I get more variability than that with each
recompile.
Mostly the change is from using f->f_... to co->co_...
ie, no difference in pointer derefs, just deref a
different pointer.
I don't see a good reason to duplicate the data.
----------------------------------------------------------------------
>Comment By: Tim Peters (tim_one)
Date: 2006-05-23 14:59
Message:
Logged In: YES
user_id=31435
Yes, it can. And did! Thanks be to Richard Jones.
----------------------------------------------------------------------
Comment By: Georg Brandl (birkenfeld)
Date: 2006-02-20 05:37
Message:
Logged In: YES
user_id=1188172
Can this go into 2.5?
----------------------------------------------------------------------
Comment By: Neal Norwitz (nnorwitz)
Date: 2005-11-02 02:14
Message:
Logged In: YES
user_id=33168
Heh, my math sucks. It should be 16 bytes, not 32. Though
I got rid of 1 more (f_restricted), so it's really 20 bytes
now. I need to clean up the patch and attach here.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1337051&group_id=5470
More information about the Patches
mailing list