-
Notifications
You must be signed in to change notification settings - Fork 436
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
Not enough unreachability/side-effects tests #1644
Comments
Agreed. The tests in unreachable.wast do not currently test for order of effects. There are quite a few tests for stack polymorphism, which are in unreached-[in]valid.wast. Your example above is unrelated to stack polymorphism, however, so I don't know what you have in mind in that regard. |
hmm we thought we had reported a stack polymorphism bug somewhere but now we don't see it either... 🤷♀️ edit: ah, it was in the exception handling proposal. |
While a real compiler should never emit code like
it would be nice to see more tests like this.
The text was updated successfully, but these errors were encountered: