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

CompileError: AsyncCompile: Wasm decoding failed: return count of 2 exceeds internal limit of 1 @+57 #123

Closed
paulyoung opened this issue Jan 15, 2019 · 4 comments

Comments

@paulyoung
Copy link
Contributor

I'm using master (c95a5e6) and getting the following error when trying to instantiate the Wasm file produced with asc --map.

Uncaught (in promise) CompileError: AsyncCompile: Wasm decoding failed: return count of 2 exceeds internal limit of 1 @+57

@paulyoung
Copy link
Contributor Author

I assume this is due to 429c7ad, and won't actually be an issue if I were to attach the debugger to a version of v8 that supported multiple return values.

This obviously doesn't work in Chrome, but will it work in our embedded version of V8?

@rossberg
Copy link
Contributor

Chrome should support this if you run it with a command line flag like

chrome --js-flags="--experimental-wasm-mv"

@nomeata
Copy link
Collaborator

nomeata commented Feb 5, 2019

Judging from the SDK demo, this is resolved.

@nomeata nomeata closed this as completed Feb 5, 2019
@paulyoung
Copy link
Contributor Author

It works when run using the flag above, yes.

knl added a commit that referenced this issue Jul 7, 2020
Various fixes in the output, to make it more correct, and avoid spamming
of GitHub histories. Should be faster, too.

This release brings the following:

Formatting:
* Fix: Add an empty line after the title. Now GitHub Commits list looks
  better.
* Fix: Remove preceeding space in the title
* Output branch name used by niv, so that it is clear what is being
  tracked
* Add a title prefix support via `title_prefix` option

Being nice:
* Reformat mentions in the changelog to reduce spamming of mentioned
  developers (now, @knl will not generate a GitHub notification when
  present in the changelog)
* Add option to avoid GitHub backreferencing the PR. That is, when we
  have #123 in the changelog, it will be a correct link to the correct
  repository, however, the referenced PR/issue will not contain a
  reference, thus reducing the spamming. See
  knl/niv-updater-action#26 for more details.
* Make the location of `sources.json` customizable

Performance:
* Replace the use of jo with jq, thus avoiding to install jo
* Do not install hub CLI, as it is present in the default image. This
  should avoid installing hub, speeding up the start.
nomeata pushed a commit that referenced this issue Jul 7, 2020
Various fixes in the output, to make it more correct, and avoid spamming
of GitHub histories. Should be faster, too.

This release brings the following:

Formatting:
* Fix: Add an empty line after the title. Now GitHub Commits list looks
  better.
* Fix: Remove preceeding space in the title
* Output branch name used by niv, so that it is clear what is being
  tracked
* Add a title prefix support via `title_prefix` option

Being nice:
* Reformat mentions in the changelog to reduce spamming of mentioned
  developers (now, @knl will not generate a GitHub notification when
  present in the changelog)
* Add option to avoid GitHub backreferencing the PR. That is, when we
  have #123 in the changelog, it will be a correct link to the correct
  repository, however, the referenced PR/issue will not contain a
  reference, thus reducing the spamming. See
  knl/niv-updater-action#26 for more details.
* Make the location of `sources.json` customizable

Performance:
* Replace the use of jo with jq, thus avoiding to install jo
* Do not install hub CLI, as it is present in the default image. This
  should avoid installing hub, speeding up the start.
dfinity-bot added a commit that referenced this issue Dec 13, 2022
## Changelog for ic-hs:
Branch: master
Commits: [dfinity/ic-hs@3cc51be5...8384593e](dfinity/ic-hs@3cc51be...8384593)

* [`9a7bcb2f`](dfinity/ic-hs@9a7bcb2) remember last trap also if on_cleanup succeeds ([dfinity/ic-hs⁠#124](https://togithub.com/dfinity/ic-hs/issues/124))
* [`8384593e`](dfinity/ic-hs@8384593) scale cycle cost of http_request calls based on subnet size ([dfinity/ic-hs⁠#123](https://togithub.com/dfinity/ic-hs/issues/123))
mergify bot pushed a commit that referenced this issue Dec 14, 2022
## Changelog for ic-hs:
Branch: master
Commits: [dfinity/ic-hs@3cc51be5...8384593e](dfinity/ic-hs@3cc51be...8384593)

* [`9a7bcb2f`](dfinity/ic-hs@9a7bcb2) remember last trap also if on_cleanup succeeds ([dfinity/ic-hs⁠#124](https://togithub.com/dfinity/ic-hs/issues/124))
* [`8384593e`](dfinity/ic-hs@8384593) scale cycle cost of http_request calls based on subnet size ([dfinity/ic-hs⁠#123](https://togithub.com/dfinity/ic-hs/issues/123))
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

No branches or pull requests

3 participants