commit 6c82cfc089466f0b3f4b61c81baee1cabece1eaa
author Damien George <damien.p.george@gmail.com> Wed Oct 04 11:31:05 2017 +1100
committer Damien George <damien.p.george@gmail.com> Wed Oct 04 11:31:05 2017 +1100
tree 66bc2b41f1a6bcdb5c0e3b20c1f6c908edd32e69
parent 9e0cdb22f1a549d5e542418a3007c756cd594b84

py/objtype: Change type of enum-to-qstr table to uint16_t to save space.

Qstr values fit in 16-bits (and this fact is used elsewhere in the code) so
no need to use more than that for the large lookup tables.  The compiler
will anyway give a warning if the qstr values don't fit in 16 bits.  Saves
around 80 bytes of code space for Thumb2 archs.

py/objtype.c

diff --git a/py/objtype.c b/py/objtype.c
index cf9311d..033765a 100644
--- a/py/objtype.c
+++ b/py/objtype.c
@@ -334,7 +334,7 @@
     return MP_OBJ_FROM_PTR(o);
 }
 
-const qstr mp_unary_op_method_name[] = {
+const uint16_t mp_unary_op_method_name[] = {
     [MP_UNARY_OP_BOOL] = MP_QSTR___bool__,
     [MP_UNARY_OP_LEN] = MP_QSTR___len__,
     [MP_UNARY_OP_HASH] = MP_QSTR___hash__,
@@ -408,7 +408,7 @@
     }
 }
 
-const qstr mp_binary_op_method_name[] = {
+const uint16_t mp_binary_op_method_name[] = {
     /*
     MP_BINARY_OP_OR,
     MP_BINARY_OP_XOR,

py/runtime.h

diff --git a/py/runtime.h b/py/runtime.h
index bbdd764..da12f8f 100644
--- a/py/runtime.h
+++ b/py/runtime.h
@@ -57,9 +57,9 @@
     mp_arg_val_t defval;
 } mp_arg_t;
 
-// defined in objtype.c
-extern const qstr mp_unary_op_method_name[];
-extern const qstr mp_binary_op_method_name[];
+// Tables mapping operator enums to qstrs, defined in objtype.c
+extern const uint16_t mp_unary_op_method_name[];
+extern const uint16_t mp_binary_op_method_name[];
 
 void mp_init(void);
 void mp_deinit(void);