New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add --enable-gc option to WAMRC #2190
Add --enable-gc option to WAMRC #2190
Conversation
e6c261d
to
75751a0
Compare
32d433d
to
e59faf7
Compare
e3ca208
to
614a4e9
Compare
wamr-compiler/main.c
Outdated
@@ -57,12 +57,13 @@ print_help() | |||
printf(" --disable-simd Disable the post-MVP 128-bit SIMD feature:\n"); | |||
printf(" currently 128-bit SIMD is supported for x86-64 and aarch64 targets,\n"); | |||
printf(" and by default it is enabled in them and disabled in other targets\n"); | |||
printf(" --disable-ref-types Disable the MVP reference types feature\n"); | |||
printf(" --disable-ref-types Disable the MVP reference types feature, it will be force disabled if GC enabled\n"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about it will be disabled forcibly
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
be force disabled
should be a grammar mistake
The runtime instance memory layout changed with GC enabled. With this patch GC enabled for WAMRC, but keep compatible for iwasm wether it enabled GC or not for non-GC wasm module.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
The runtime instance memory layout changed with GC enabled. With this patch GC enabled for WAMRC, but keep compatible for iwasm wether it enabled GC or not for non-GC wasm module.
It may waste some memory for iwasm without GC support since the GC relative fileds are always here, let's do optimization until
AOT fully support GC.