-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Emscripting Python with optimizations+assertions produces invalid code #28
Comments
Thanks! That was an old assertion that is no longer needed. The bug only happened with that combination of those flags, which we don't test (too many combinations to check everything...). Fixed in 009dc48 |
tlively
pushed a commit
to tlively/emscripten
that referenced
this issue
Mar 23, 2022
Add support for 'mode' parameter in createSyncAccessHandle
steveisok
pushed a commit
to steveisok/emscripten
that referenced
this issue
Sep 18, 2023
…913.1 (emscripten-core#28) [dotnet/release/8.0] Update dependencies from dotnet/arcade
eagleoflqj
pushed a commit
to eagleoflqj/emscripten
that referenced
this issue
Nov 16, 2024
Add fcitx4's show input method status when focus in.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Running emscripten on
python.ll
withOPTIMIZE = 1
,RELOOP=1
andASSERTIONS=1
produces many invalid blocks, of the general form:The extra else is added at jsifier.js:402. Removing these lines from the result produces a working file (which is about 10 times faster than the unoptimized version).
The text was updated successfully, but these errors were encountered: