Skip to content

Updated multi-value message#1797

Merged
sbc100 merged 2 commits intoWebAssembly:mainfrom
jzabinski-dolios:multi-value-msg
Jan 10, 2022
Merged

Updated multi-value message#1797
sbc100 merged 2 commits intoWebAssembly:mainfrom
jzabinski-dolios:multi-value-msg

Conversation

@jzabinski-dolios
Copy link
Contributor

Resolves #1796

To test locally:

  1. Make a WAT file containing a function which returns multiple values. Example:
(module
    (func $mult (export "mult")
        (result f64 f64)
        f64.const 1
        f64.const 2
    )
)
  1. Use wat2wasm to compile, disabling multi-value. For example:
sudo bin/wat2wasm --disable-multi-value examples/multi.wat -o examples/multi.wasm

An error should be returned by the compiler. The wording of the error should be multiple result values are not supported without multi-value enabled.

@sbc100
Copy link
Member

sbc100 commented Jan 8, 2022

Looks like there is lint failure.. can you git clang-format origin/main?

@sbc100 sbc100 enabled auto-merge (squash) January 10, 2022 15:45
@sbc100 sbc100 merged commit 3cd653a into WebAssembly:main Jan 10, 2022
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.

Wording of error message 'multiple result values not currently supported'

2 participants