Skip to content

Add support for initializing/getting/setting funcrefs in GC structs from host APIs#9454

Merged
alexcrichton merged 2 commits into
bytecodealliance:mainfrom
fitzgen:funcrefs-in-host-gc-object-apis
Oct 11, 2024
Merged

Add support for initializing/getting/setting funcrefs in GC structs from host APIs#9454
alexcrichton merged 2 commits into
bytecodealliance:mainfrom
fitzgen:funcrefs-in-host-gc-object-apis

Conversation

@fitzgen
Copy link
Copy Markdown
Member

@fitzgen fitzgen commented Oct 10, 2024

We implemented support for funcrefs in arrays in both compiled Wasm and host APIs, but somehow only implemented support for funcrefs in structs for compiled Wasm code, and mistakenly forgot about them in host APIs (and things that use host APIs, such as const expressions).

@fitzgen fitzgen requested a review from a team as a code owner October 10, 2024 23:50
@fitzgen fitzgen requested review from alexcrichton and removed request for a team October 10, 2024 23:50
… from host APIs

We implemented support for `funcref`s in both Wasm and host APIs for arrays, but
somehow only implemented support for `funcref`s in structs for compiled Wasm
code, and mistakenly forgot about them for structs and host APIs (and things
that use host APIs, such as const expressions).
@fitzgen fitzgen force-pushed the funcrefs-in-host-gc-object-apis branch from 2810a29 to d5abe0b Compare October 10, 2024 23:52
@alexcrichton alexcrichton added this pull request to the merge queue Oct 11, 2024
Merged via the queue into bytecodealliance:main with commit 9529243 Oct 11, 2024
@fitzgen fitzgen deleted the funcrefs-in-host-gc-object-apis branch October 11, 2024 15:33
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