diff options
| author | JF Bastien <jfb@chromium.org> | 2016-02-05 05:11:50 -0800 |
|---|---|---|
| committer | JF Bastien <jfb@chromium.org> | 2016-02-05 05:11:50 -0800 |
| commit | 7459e5af01fbe3a8e75e73783794b4cdffda34e9 (patch) | |
| tree | aae2c28eb1cc465a4582208d6c713584a51a1bcf /test/unit.wast | |
| parent | 8f67b6e27a38c93fbca7f3c44a88889b3896952f (diff) | |
| download | binaryen-7459e5af01fbe3a8e75e73783794b4cdffda34e9.tar.gz binaryen-7459e5af01fbe3a8e75e73783794b4cdffda34e9.tar.bz2 binaryen-7459e5af01fbe3a8e75e73783794b4cdffda34e9.zip | |
Fix select
The ordering changed in: https://github.com/WebAssembly/spec/pull/221
Which changed the spec tests, breaking sexpr-wasm because it pulls in the spec tests. This was then fixed:
https://github.com/WebAssembly/sexpr-wasm-prototype/commit/23dc368148fc7827a603e3853f5a40287eb9effe
Which in turn breaks when binaryen feeds sexpr-wasm .wast files with the old select operand ordering.
Note that this PR has new failures when running the torture tests in binaryen-shell: the order of evaluation is correct in binaryen-shell but isn't emitted properly by LLVM in the .s files. This will require another patch to fix LLVM.
Diffstat (limited to 'test/unit.wast')
| -rw-r--r-- | test/unit.wast | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/test/unit.wast b/test/unit.wast index cde4722ef..928c30157 100644 --- a/test/unit.wast +++ b/test/unit.wast @@ -311,15 +311,15 @@ (i32.const 0) ) (i32.select - (i32.lt_s - (get_local $asm2wasm_i32_temp) - (i32.const 0) - ) (i32.sub (i32.const 0) (get_local $asm2wasm_i32_temp) ) (get_local $asm2wasm_i32_temp) + (i32.lt_s + (get_local $asm2wasm_i32_temp) + (i32.const 0) + ) ) ) ) |