py: Pass in address to compiled module instead of returning it.

This change makes it so the compiler and persistent code loader take a
mp_compiled_module_t* as their last argument, instead of returning this
struct.  This eliminates a duplicate context variable for all callers of
these functions (because the context is now stored in the
mp_compiled_module_t by the caller), and also eliminates any confusion
about which context to use after the mp_compile_to_raw_code or
mp_raw_code_load function returns (because there is now only one context,
that stored in mp_compiled_module_t.context).

Reduces code size by 16 bytes on ARM Cortex-based ports.

Signed-off-by: Damien George <damien@micropython.org>

1 files changed, 1 insertions, 1 deletions

diff --git a/py/bc.h b/py/bc.h
index 6350eee52..0e23255f7 100644
--- a/py/bc.h
+++ b/py/bc.h
@@ -214,7 +214,7 @@ typedef struct _mp_module_context_t {
 
 // Outer level struct defining a compiled module.
 typedef struct _mp_compiled_module_t {
-    const mp_module_context_t *context;
+    mp_module_context_t *context;
     const struct _mp_raw_code_t *rc;
     #if MICROPY_PERSISTENT_CODE_SAVE
     bool has_native;