exception-handling
exception-handling copied to clipboard
[legacy] Add setjmp bait
sadly UBSAN doesn't appear to detect this (yet).
Shouldn't we have a corresponding test for the non-legacy instruction?
yes, eventually
wasm2c only supports EHL at this point so having the test for EHv4 isn't that critical yet.
surprisingly, this test also appears to break wabt interpreter.
for context, the interpreter isn't even using setjmp.
Sorry I may lack some context, but what is this test about? They say setjmp and longjmp but here they are just normal functions. Does this test have any specific characteristics or does this just happen to be the case the wabt interpreter fails on? Apparenly this PR doesn't have any changes to the spec interpreter so that is fine, right?
the test targets wasm2c specifically, where exceptions are implemented using setjmp/longjmp, but also happens to trigger a bug in wabt interpreter (which is mostly unrelated to wasm2c, despite also being part of wabt).
the wabt interpreter is unrelated to the spec interpreter, the spec interpreter is probably fine.