Skip to content
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

Fix Wasm VM Initialization Bug #43

Merged
merged 5 commits into from
Dec 16, 2024
Merged

Conversation

sskeirik
Copy link

This PR fixes a bug with Wasm VM initialization in the remote case by:

  1. Building the binary header for use by the ULM.decode function
  2. Changing the VM initialization to load the entrypoint correctly in each case

Also, an extraneous comment was deleted.

@virgil-serbanuta virgil-serbanuta merged commit ae2c6f3 into rust_minimize Dec 16, 2024
2 checks passed
@virgil-serbanuta virgil-serbanuta deleted the build_rich_header branch December 16, 2024 17:17
virgil-serbanuta added a commit that referenced this pull request Dec 16, 2024
* add helper scripts

* Build fixes and instructions for the remote ULM demo

* improve pykwasm scripts

* update scripts

* fix sorts

* add formatting

* update deploy_contract script

* add fix to deploy contract script

* fix issues in python scripts

* improve README

* update README further

* remove files to be committed in a separate PR

* remove unfinished script

* remove redundant changes

* improve deploy contract script

* apply formatter

* remove deprecated deploy contract script

* add arity check

* update rust build

* improve README

* Fix Wasm VM Initialization Bug (#43)

* add kore rich header build

* update entrypoint resolution

* fix syntax error

* fix more errors

* remove deprecated comment

---------

Co-authored-by: Virgil <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants