Commit 3a4dfc87 authored by Guido van Rossum's avatar Guido van Rossum

Another ugly inlining hack, expanding the two PyDict_GetItem() calls

in LOAD_GLOBAL.  Besides saving a C function call, it saves checks
whether f_globals and f_builtins are dicts, and extracting and testing
the string object's hash code is done only once.  We bail out of the
inlining if the name is not exactly a string, or when its hash is -1;
because of interning, neither should ever happen.  I believe interning
guarantees that the hash code is set, and I believe that the 'names'
tuple of a code object always contains interned strings, but I'm not
assuming that -- I'm simply testing hash != -1.

On my home machine, this makes a pystone variant with new-style
classes and slots run at the same speed as classic pystone!  (With
new-style classes but without slots, it is still a lot slower.)
parent e3a8e7ed
...@@ -1709,13 +1709,37 @@ eval_frame(PyFrameObject *f) ...@@ -1709,13 +1709,37 @@ eval_frame(PyFrameObject *f)
case LOAD_GLOBAL: case LOAD_GLOBAL:
w = GETITEM(names, oparg); w = GETITEM(names, oparg);
if (PyString_CheckExact(w)) {
long hash = ((PyStringObject *)w)->ob_shash;
if (hash != -1) {
/* Inline the PyDict_GetItem() calls */
PyDictObject *d;
d = (PyDictObject *)(f->f_globals);
x = d->ma_lookup(d, w, hash)->me_value;
if (x != NULL) {
Py_INCREF(x);
PUSH(x);
continue;
}
d = (PyDictObject *)(f->f_builtins);
x = d->ma_lookup(d, w, hash)->me_value;
if (x != NULL) {
Py_INCREF(x);
PUSH(x);
continue;
}
goto load_global_error;
}
}
/* This is the un-inlined version of the code above */
x = PyDict_GetItem(f->f_globals, w); x = PyDict_GetItem(f->f_globals, w);
if (x == NULL) { if (x == NULL) {
x = PyDict_GetItem(f->f_builtins, w); x = PyDict_GetItem(f->f_builtins, w);
if (x == NULL) { if (x == NULL) {
load_global_error:
format_exc_check_arg( format_exc_check_arg(
PyExc_NameError, PyExc_NameError,
GLOBAL_NAME_ERROR_MSG ,w); GLOBAL_NAME_ERROR_MSG, w);
break; break;
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment