diff options
author | Alon Zakai <azakai@google.com> | 2021-03-24 16:40:00 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-03-24 16:40:00 -0700 |
commit | 55af054b6b0b6d30fca7da43ac1c4522ff812e67 (patch) | |
tree | db90a7b078ec89ac234fb1ba9f7e75759cec4430 /src/wasm/wasm-s-parser.cpp | |
parent | 17684a20196194d6b0a930b4bd10688f9e2d3fa8 (diff) | |
download | binaryen-55af054b6b0b6d30fca7da43ac1c4522ff812e67.tar.gz binaryen-55af054b6b0b6d30fca7da43ac1c4522ff812e67.tar.bz2 binaryen-55af054b6b0b6d30fca7da43ac1c4522ff812e67.zip |
Validator: Pass the module along when printing errors, so type names are used (#3727)
For example, on this invalid wat:
(module
(type $vec (struct (field i64)))
(func $test
(drop
(struct.new_with_rtt $vec (i32.const 1) (rtt.canon $vec))
)
)
)
We used to print:
[wasm-validator error in function test] struct.new operand must have proper type, on
(struct.new_with_rtt ${i64}
(i32.const 1)
(rtt.canon ${i64})
)
We will now print:
[wasm-validator error in function test] struct.new operand must have proper type, on
(struct.new_with_rtt $vec
(i32.const 1)
(rtt.canon $vec)
)
Note that $vec is used. In real-world examples the autogenerated structural name
can be huge, which this avoids.
Diffstat (limited to 'src/wasm/wasm-s-parser.cpp')
0 files changed, 0 insertions, 0 deletions