Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dust off the fuzzers, update dependencies and use wasm-smith. #2186

Merged
merged 8 commits into from
Mar 16, 2021

Conversation

nlewycky
Copy link
Contributor

Switch compilation tests to wasm-smith which always generates valid modules.

Update to newer libfuzzer-sys and use a versioned crates instead of a git repository.

Delete "headless_cranelift.rs" which is an older copy of native_cranelift.rs and wasn't referenced in the Cargo.toml.

…odules.

Update to newer libfuzzer-sys and use a versioned crates instead of a git repository.

Delete "headless_cranelift.rs" which is an older copy of native_cranelift.rs and wasn't referenced in the Cargo.toml.
@nlewycky nlewycky requested a review from syrusakbary as a code owner March 15, 2021 18:55
This wasm module is valid:
```
(module
  (memory (;0;) 0)
  (data (;0;) (i32.const 34) ""))
```
and we want to just ignore that problem.
Detect and ignore more errors from singlepass about multi-value returns.
The start function can contain unreachable instructions which triggers a trap and looks like instantiation failed. Instantiation did fail, but not for reasons we're interested in.
@syrusakbary
Copy link
Member

bors r+

@bors bors bot merged commit 513571d into master Mar 16, 2021
@bors bors bot deleted the feature/update-fuzzers branch March 16, 2021 10:45
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.

2 participants