Conversation
keithw
left a comment
There was a problem hiding this comment.
lgtm. Thank you for working on this! Could you please submit the test to the exception-handling repo so it can get picked up by the testsuite repo and included in the WABT testsuite (#2203 (comment))?
Alternately, I think it would be fine to amend this PR to include that as a one-off test if you just want this to get merged ASAP, and we can merge the upstream testsuite fix down the road. Either way it would be nice to make sure this scenario gets tested so different implementations all agree on the behavior of mixing br with try/catch.
keithw
left a comment
There was a problem hiding this comment.
lgtm. Will be great when the upstream repo includes some tests of br interacting with try/catch. Thank you for testing and fixing this.
(didn't test locally but should work)
Fixes #2203