Fix asan for jl_precompile_toplevel_module (JuliaLang#43885)

PR JuliaLang#43793 passed the buildkite test but the logs for JuliaLang#43881 show an address sanitzer failure. Removing jl_precompile_toplevel_module from jl_exported_data.inc fixes the error. For good measure, set it to NULL at the point of definition, even though it gets nulled during initialization.
inkydragon · Jan 21, 2022 · a327428 · a327428
1 parent 5f79cb3
commit a327428
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/src/jl_exported_data.inc b/src/jl_exported_data.inc
@@ -85,7 +85,6 @@
  XX(jl_pinode_type) \
  XX(jl_pointer_type) \
  XX(jl_pointer_typename) \
- XX(jl_precompile_toplevel_module) \
  XX(jl_quotenode_type) \
  XX(jl_readonlymemory_exception) \
  XX(jl_ref_type) \

diff --git a/src/toplevel.c b/src/toplevel.c
@@ -34,7 +34,7 @@ htable_t jl_current_modules;
 jl_mutex_t jl_modules_mutex;
 
 // During incremental compilation, the following gets set
-JL_DLLEXPORT jl_module_t *jl_precompile_toplevel_module;  // the toplevel module currently being defined
+JL_DLLEXPORT jl_module_t *jl_precompile_toplevel_module = NULL; // the toplevel module currently being defined
 
 JL_DLLEXPORT void jl_add_standard_imports(jl_module_t *m)
 {