diff options
| author | Alon Zakai <alonzakai@gmail.com> | 2018-09-11 12:08:14 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-09-11 12:08:14 -0700 |
| commit | 249b4b78d765c0f83029c3afd107cbe9ea025681 (patch) | |
| tree | 53b4bea406f7134a0b1bde8dda5291ad07a863fd /test/binaryen.js | |
| parent | f831369f8586f86cafe10ee4f34c9b1f239abbfc (diff) | |
| download | binaryen-249b4b78d765c0f83029c3afd107cbe9ea025681.tar.gz binaryen-249b4b78d765c0f83029c3afd107cbe9ea025681.tar.bz2 binaryen-249b4b78d765c0f83029c3afd107cbe9ea025681.zip | |
Binary format local parsing fixes (#1664)
* Error if there are more locals than browsers allow (50,000). We usually just warn about stuff like this, but we do need some limit (or else we hang or OOM), and if so, why not use the agreed-upon Web limit.
* Do not generate nice string names for locals in binary parsing - the name is just $var$x instead of $x, so not much benefit, and worse as our names are interned this is actually slow (which is why the fuzz testcase here hangs instead of OOMing).
Testcases and bugreport in #1663.
Diffstat (limited to 'test/binaryen.js')
| -rw-r--r-- | test/binaryen.js/kitchen-sink.js.txt | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/test/binaryen.js/kitchen-sink.js.txt b/test/binaryen.js/kitchen-sink.js.txt index 5edad3d88..2bec81e91 100644 --- a/test/binaryen.js/kitchen-sink.js.txt +++ b/test/binaryen.js/kitchen-sink.js.txt @@ -1067,10 +1067,10 @@ optimized: module loaded from binary form: (module (type $0 (func (param i32 i32) (result i32))) - (func $adder (; 0 ;) (type $0) (param $var$0 i32) (param $var$1 i32) (result i32) + (func $adder (; 0 ;) (type $0) (param $0 i32) (param $1 i32) (result i32) (i32.add - (get_local $var$0) - (get_local $var$1) + (get_local $0) + (get_local $1) ) ) ) |