[Patches] [ python-Patches-1337051 ] remove 4 ints from PyFrameObject

[SourceForge.net]

Patches item #1337051, was opened at 2005-10-25 08:18
Message generated for change (Comment added) made by birkenfeld
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: Open
Resolution: None
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: Georg Brandl (birkenfeld)
Date: 2006-02-20 11:37

Message:
Logged In: YES 
user_id=1188172

Can this go into 2.5?

----------------------------------------------------------------------

Comment By: Neal Norwitz (nnorwitz)
Date: 2005-11-02 08: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