Serhiy Storchaka added the comment:

When LOAD_NAME is generated? Is it worth to optimize this case? Aren't 
LOAD_FAST and LOAD_GLOBAL used in performance critical code?

It looks to me that there is a bug in fast path of _PyDict_LoadGlobal. If the 
first lookup fails, it can raise an exception. We have to add

    if (PyErr_Occurred())
        return NULL;

before the second lookup.

Just for reference, the fast path for LOAD_GLOBAL was added in 8f8fe990e82c.

----------
nosy: +benjamin.peterson, rhettinger

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue25557>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to