Skip to content
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

Noexn type #297

Closed
thibaudmichaud opened this issue Feb 19, 2024 · 4 comments
Closed

Noexn type #297

thibaudmichaud opened this issue Feb 19, 2024 · 4 comments

Comments

@thibaudmichaud
Copy link
Collaborator

#282 included the notion of a noexn type to mirror the other null types of the GC proposal, but this was not added in the spec yet.
Should it be added, and do we already have an opcode for it?

@aheejin
Copy link
Member

aheejin commented Feb 20, 2024

I think we should add the opcode. Binaryen also needed it to implement EH. I currently encode it as 0xff for a placeholder: https://github.com/WebAssembly/binaryen/blob/feb8f24128c0c3bd53862b2f39acc5116f8ae87e/src/wasm-binary.h#L478

@rossberg
Copy link
Member

Right, this is another example of some feature falling into the intersection of two proposals. Since GC has advanced to phase 4 now, it should be added here. (At least in the overview, in the actual spec we'll need to wait until we rebase on the spec including GC.)

@rossberg
Copy link
Member

Please see #298.

@aheejin
Copy link
Member

aheejin commented Feb 21, 2024

Closed by #298.

@aheejin aheejin closed this as completed Feb 21, 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

No branches or pull requests

3 participants