Skip to content

Conversation

@wenyongh
Copy link
Contributor

Currently when calling wasm_runtime_call_wasm() to invoke wasm function
with externref type argument from runtime embedder, developer needs to
use wasm_externref_obj2ref() to convert externref obj into an internal ref
index firstly, which is not convenient to developer.
To align with GC feature in which all the references passed to
wasm_runtime_call_wasm() can be object pointers directly, we change the
interface of wasm_runtime_call_wasm() to allow to pass object pointer
directly for the externref argument, and refactor the related codes, update
the related samples and the document.

wenyongh and others added 9 commits December 7, 2021 10:27
Currently when calling wasm_runtime_call_wasm() to invoke wasm function
with externref type argument from runtime embedder, developer need to
use wasm_externref_obj2ref() to convert externref obj to an internal ref index
firstly, which is not convenient to developer.
To align with GC feature in which all the references passed to
wasm_runtime_call_wasm() can be object pointer directly, we change the
interface of wasm_runtime_call_wasm() to allow to pass object pointer
directly for the externref argument, and refactor the related codes, update
the related samples and the document.
Using signature string instead as only native function can introduce
it when it is registered.
Merge main into dev/refactor_ref_types
Merge main into dev/refactor_ref_types
Merge main into dev/refactor_ref_types
Specify a commit ID for XNNPACK workload (#966)
@wenyongh wenyongh merged commit 260d36a into main Jan 19, 2022
@wenyongh wenyongh deleted the dev/refactor_ref_types branch January 21, 2022 03:10
xujuntwt95329 pushed a commit to xujuntwt95329/wasm-micro-runtime that referenced this pull request Jan 24, 2022
…lliance#971)

Currently when calling wasm_runtime_call_wasm() to invoke wasm function
with externref type argument from runtime embedder, developer needs to
use wasm_externref_obj2ref() to convert externref obj into an internal ref
index firstly, which is not convenient to developer.
To align with GC feature in which all the references passed to
wasm_runtime_call_wasm() can be object pointers directly, we change the
interface of wasm_runtime_call_wasm() to allow to pass object pointer
directly for the externref argument, and refactor the related codes, update
the related samples and the document.
yamt added a commit to yamt/wasm-micro-runtime that referenced this pull request Aug 19, 2022
It's unused since the following commit.
```
commit 260d36a
Author: Wenyong Huang <wenyong.huang@intel.com>
Date:   Wed Jan 19 11:25:08 2022 +0800

    Refactor externref related APIs of reference types feature (bytecodealliance#971)

    Currently when calling wasm_runtime_call_wasm() to invoke wasm function
    with externref type argument from runtime embedder, developer needs to
    use wasm_externref_obj2ref() to convert externref obj into an internal ref
    index firstly, which is not convenient to developer.
    To align with GC feature in which all the references passed to
    wasm_runtime_call_wasm() can be object pointers directly, we change the
    interface of wasm_runtime_call_wasm() to allow to pass object pointer
    directly for the externref argument, and refactor the related codes, update
    the related samples and the document.
```
vickiegpt pushed a commit to vickiegpt/wamr-aot-gc-checkpoint-restore that referenced this pull request May 27, 2024
…lliance#971)

Currently when calling wasm_runtime_call_wasm() to invoke wasm function
with externref type argument from runtime embedder, developer needs to
use wasm_externref_obj2ref() to convert externref obj into an internal ref
index firstly, which is not convenient to developer.
To align with GC feature in which all the references passed to
wasm_runtime_call_wasm() can be object pointers directly, we change the
interface of wasm_runtime_call_wasm() to allow to pass object pointer
directly for the externref argument, and refactor the related codes, update
the related samples and the document.
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