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
Dev/gc refactor 3 #2487
Dev/gc refactor 3 #2487
Conversation
d356bdf
to
d0e0b15
Compare
core/iwasm/compilation/aot_emit_gc.c
Outdated
check_array_obj_succ)) | ||
goto fail; | ||
|
||
if (!aot_get_array_obj_length(comp_ctx, func_ctx, array_obj, &array_len)) |
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.
Don't call function, just generate LLVM IR of array_obj->length >> WASM_ARRAY_LENGTH_SHIFT
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.
This static function generates LLVM IR(the equivalent of array_obj->length >> WASM_ARRAY_LENGTH_SHIFT
) rather than calling a function, it's used in 3 places so I made it into a function.
d0e0b15
to
cf8a668
Compare
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
…ated new opcodes This reverts commit 1c0b549
…apper call, load/store directly
86a7aeb
to
ee92031
Compare
The first draft of the second part of AOT GC compilation opcodes, the struct-related and array-related new opcodes:
After merging the second part, will rebase accordingly