commit a0d32991ed57ebbbb8c9207dab3223b12ca4aa44
author Paul Sokolovsky <pfalcon@users.sourceforge.net> Sat Apr 05 04:51:26 2014 +0300
committer Paul Sokolovsky <pfalcon@users.sourceforge.net> Sat Apr 05 05:10:12 2014 +0300
tree 21197e7430836f4e665546f35de19d5a5474eca7
parent e3f58c838022fcdbb39b9a28274b72bc0f69d1ea

mp_load_name(): Optimize for outer scope where locals == globals.

py/runtime.c

diff --git a/py/runtime.c b/py/runtime.c
index a5f45a2..238c3a5 100644
--- a/py/runtime.c
+++ b/py/runtime.c
@@ -93,13 +93,15 @@
 
 mp_obj_t mp_load_name(qstr qstr) {
     // logic: search locals, globals, builtins
-    DEBUG_OP_printf("load name %s\n", qstr_str(qstr));
-    mp_map_elem_t *elem = mp_map_lookup(map_locals, MP_OBJ_NEW_QSTR(qstr), MP_MAP_LOOKUP);
-    if (elem != NULL) {
-        return elem->value;
-    } else {
-        return mp_load_global(qstr);
+    DEBUG_OP_printf("load name %s\n", map_locals, qstr_str(qstr));
+    // If we're at the outer scope (locals == globals), dispatch to load_global right away
+    if (map_locals != map_globals) {
+        mp_map_elem_t *elem = mp_map_lookup(map_locals, MP_OBJ_NEW_QSTR(qstr), MP_MAP_LOOKUP);
+        if (elem != NULL) {
+            return elem->value;
+        }
     }
+    return mp_load_global(qstr);
 }
 
 mp_obj_t mp_load_global(qstr qstr) {