Skip to content

Conversation

@lum1n0us
Copy link
Collaborator

@lum1n0us lum1n0us commented Jan 4, 2023

No description provided.

@lum1n0us lum1n0us force-pushed the quit_if_meet_unimport branch 2 times, most recently from d63314b to 7cf9829 Compare January 5, 2023 09:55
@lum1n0us lum1n0us changed the title Quit if meeting unimport functions or globals Quit if meeting unimport globals Jan 5, 2023
@lum1n0us lum1n0us marked this pull request as ready for review January 5, 2023 09:56
Comment on lines 989 to 990
LOG_WARNING("warning: failed to link import global (%s, %s)",
global->module_name, global->global_name);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be an error here, use set_error_buf

Comment on lines 1254 to 1255
LOG_WARNING("warning: failed to link import global (%s, %s)",
global->module_name, global->field_name);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same as above, should be an error

There are warning for unimport functions
@lum1n0us lum1n0us force-pushed the quit_if_meet_unimport branch from 7cf9829 to 7308d80 Compare January 6, 2023 04:53
@wenyongh wenyongh merged commit 7401718 into bytecodealliance:main Jan 6, 2023
@lum1n0us lum1n0us deleted the quit_if_meet_unimport branch January 9, 2023 01:24
NingW101 pushed a commit to NingW101/wasm-micro-runtime that referenced this pull request Jan 11, 2023
vickiegpt pushed a commit to vickiegpt/wamr-aot-gc-checkpoint-restore that referenced this pull request May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants