TOPLLab / WARDuino

📟 A dynamic WebAssembly VM for embedded systems
https://topllab.github.io/WARDuino/
Mozilla Public License 2.0
73 stars 7 forks source link

:microscope: Load tidx for callbacks from memory #197

Open tolauwae opened 12 months ago

carllocos commented 12 months ago

By any chance is this PR meant to solve an asc compilation issue related to callbacks?

In my case, I have an Assemblyscript application for which the asc compiler generates a tidx out of range https://github.com/TOPLLab/WARDuino/blob/4ce14c2457bf8fedb091de012b4563315304326a/src/Primitives/emulated.cpp#L448

The same issue also happens on the MCU.

If not, I'll create a separate issue and I will fix the issue there via another PR.

tolauwae commented 12 months ago

Yes. This has been a known issue for a year, and I finally got round to fixing it. This PR is not ready to be merged.

tolauwae commented 12 months ago

This has been broken since #73, but the behaviour/reason for the failure has changed since.

carllocos commented 12 months ago

I confirm what you describe in issue #73 is what I have been experiencing.