From f4e84654565bb510f332ce3fb0cdce086574af3a Mon Sep 17 00:00:00 2001 From: Jameson Nash Date: Tue, 20 Feb 2024 17:02:26 +0000 Subject: [PATCH] fix sysimage-native-code=yes option Follow up to #53373, it seems this assert was broken for empty packages, causing CI issues. It is not necessary. --- src/staticdata.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/src/staticdata.c b/src/staticdata.c index abcdd76dac507..eab19607d9eba 100644 --- a/src/staticdata.c +++ b/src/staticdata.c @@ -3092,11 +3092,7 @@ static void jl_restore_system_image_from_stream_(ios_t *f, jl_image_t *image, jl // in --build mode only use sysimg data, not precompiled native code int imaging_mode = jl_generating_output() && !jl_options.incremental; - if (!imaging_mode && jl_options.use_sysimage_native_code == JL_OPTIONS_USE_SYSIMAGE_NATIVE_CODE_YES) { - if (image->gvars_base) - assert(image->fptrs.ptrs); - } - else { + if (imaging_mode || jl_options.use_sysimage_native_code != JL_OPTIONS_USE_SYSIMAGE_NATIVE_CODE_YES) { memset(&image->fptrs, 0, sizeof(image->fptrs)); image->gvars_base = NULL; }