Use a pointer for PC instead of an instruction counter

Also mark code in modules as const. This optimization brings an additional 20% speed upgrade to the Sudoku benchmark. Signed-off-by: Paul Guyot <[email protected]>
atomvm · Aug 21, 2023 · fd62240 · fd62240
1 parent 2706a1d
commit fd62240
Show file tree

Hide file tree

Showing 4 changed files with 822 additions and 1,384 deletions.
diff --git a/src/libAtomVM/module.c b/src/libAtomVM/module.c
@@ -60,7 +60,7 @@
     static void *module_uncompress_literals(const uint8_t *litT, int size);
 #endif
 static struct LiteralEntry *module_build_literals_table(const void *literalsBuf);
-static void module_add_label(Module *mod, int index, void *ptr);
+static void module_add_label(Module *mod, int index, const uint8_t *ptr);
 static enum ModuleLoadResult module_build_imported_functions_table(Module *this_module, uint8_t *table_data, GlobalContext *glb);
 static void parse_line_table(uint16_t **line_refs, struct ModuleFilename **filenames, uint8_t *data, size_t len);
 
@@ -235,7 +235,7 @@ term module_get_exported_functions(Module *this_module, Heap *heap, GlobalContex
     return result_list;
 }
 
-static void module_add_label(Module *mod, int index, void *ptr)
+static void module_add_label(Module *mod, int index, const uint8_t *ptr)
 {
     mod->labels[index] = ptr;
 }

diff --git a/src/libAtomVM/module.h b/src/libAtomVM/module.h
@@ -114,7 +114,7 @@ struct Module
 
     const struct ExportedFunction **imported_funcs;
 
-    void **labels;
+    const uint8_t **labels;
 
     void *literals_data;