-
-
Notifications
You must be signed in to change notification settings - Fork 3
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
All errors become WASM traps #2
Comments
What's the |
Ah, sorry, I didn't your comment carefully. That's expected behavior at the moment. Until WASM has support for C++ exceptions, all errors will end up in WASM traps. |
The original error is easily worked around, however it seems that yosys/nextpnr often generate C++ exceptions. For example, I ran nmigen's |
Yep. Until WASM gets first-class exceptions I'm afraid that's what's going to happen. I tried to get it a bit better where possible, but the amount of hacks I'm willing to add (and the upstream is willing to accept) is limited... |
A note for anyone else finding that WASM traps are constantly generated. In my previous comment, it seemed that C++ exceptions were generated very frequently, and I found that yowasp-nextpnr was unusable from nmigen. This was due to installing nmigen from pip (right now the pip version is ~5 months old). Once I updated to the git version of nmigen, and now that YoWASP/yosys#1 is fixed, nmigen's blinky example now runs and uploads fine, rather than crashing during the writing of the blif file. |
Ah, interesting! I indeed have not tested with nmigen 0.2. Nmigen 0.3 is going to be released soon, so that should not be a problem anymore after that. |
This will be fixed when WebAssembly/wasi-sdk#198 is merged and exception handling support lands in wasmtime. |
Thanks for all the hard work making this package available.
I installed only one package with the command
pip install yowasp-nextpnr-ice40-8k
, on Windows 10 x64, Anaconda Python 3.7 (same result for 3.8). It installsyowasp_nextpnr_ice40_8k-0.0.post2625.dev9-py3-none-any.whl
I then run
yowasp-nextpnr-ice40
. It prints out the full list of command line options, then prints out the following error message:where
<my home>
represents my windows home folder.If I run
pip install yowasp-nextpnr-ice40-all
, then the error no longer occurs.The text was updated successfully, but these errors were encountered: