Stream: git-wasmtime

Topic: wasmtime / PR #5128 Cranelift: disable egraphs in fuzzing...


view this post on Zulip Wasmtime GitHub notifications bot (Oct 25 2022 at 22:54):

cfallin opened PR #5128 from disable-egraph-fuzzing to main:

As per [this comment], with a few recent discussions it's become clear that we want to refactor egraphs in a way that will subsume, or make irrelevant, some of the recent fuzzbugs that have arisen (and likely lead to others, which we'll want to fix!). Rather than chase these down then refactor later, it probably makes sense not to spend the human time or fuzzing time doing so. This PR turns off egraphs support in fuzzing configurations for now, to be re-enabled later.

[this comment]: https://github.com/bytecodealliance/wasmtime/issues/5126#issuecomment-1291222515

<!--

Please ensure that the following steps are all taken care of before submitting
the PR.

Please ensure all communication adheres to the code of conduct.
-->

view this post on Zulip Wasmtime GitHub notifications bot (Oct 25 2022 at 22:54):

cfallin requested elliottt for a review on PR #5128.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 25 2022 at 22:54):

cfallin requested fitzgen for a review on PR #5128.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 25 2022 at 22:54):

cfallin requested jameysharp for a review on PR #5128.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 25 2022 at 23:04):

fitzgen submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 25 2022 at 23:07):

cfallin updated PR #5128 from disable-egraph-fuzzing to main.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 25 2022 at 23:08):

elliottt submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 25 2022 at 23:09):

cfallin has enabled auto merge for PR #5128.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 25 2022 at 23:51):

cfallin merged PR #5128.


Last updated: Nov 22 2024 at 17:03 UTC