Skip to content

Conversation

@aheejin
Copy link
Member

@aheejin aheejin commented Oct 20, 2024

@aheejin aheejin requested a review from sbc100 October 20, 2024 07:07
@aheejin
Copy link
Member Author

aheejin commented Oct 20, 2024

cc @SoniEx2

@SoniEx2
Copy link
Collaborator

SoniEx2 commented Oct 20, 2024

uh wait what? we thought we pulled that from the spec, not the explainer... (but we may also have misread something)

@SoniEx2
Copy link
Collaborator

SoniEx2 commented Oct 20, 2024

oh huh that is in the spec... well, anyway, LGTM.

@sbc100 sbc100 merged commit d5cb812 into WebAssembly:main Oct 21, 2024
@aheejin aheejin deleted the exnref_opcode branch October 22, 2024 04:05
@SoniEx2
Copy link
Collaborator

SoniEx2 commented Nov 30, 2024

added this to the (long) list of missing upstream tests at WebAssembly/exception-handling#333

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.

3 participants