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/reduce | |
| 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/reduce')
| -rw-r--r-- | test/reduce/destructive.wast.txt | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/reduce/destructive.wast.txt b/test/reduce/destructive.wast.txt index 05a3d3118..8721c051f 100644 --- a/test/reduce/destructive.wast.txt +++ b/test/reduce/destructive.wast.txt @@ -1,7 +1,7 @@ (module (type $0 (func (param i32) (result i32))) (export "x" (func $0)) - (func $0 (; 0 ;) (type $0) (param $var$0 i32) (result i32) + (func $0 (; 0 ;) (type $0) (param $0 i32) (result i32) (i32.const 100) ) ) |