Stream: git-wasmtime

Topic: wasmtime / PR #4845 Various improvements to differential ...


view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2022 at 03:12):

alexcrichton opened PR #4845 from improve-differential-fuzzing to main:

I separated out a few commits below to the differential fuzzer which I was working on recently. The main changes are:

view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2022 at 03:12):

alexcrichton requested abrown for a review on PR #4845.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2022 at 14:03):

alexcrichton updated PR #4845 from improve-differential-fuzzing to main.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2022 at 16:43):

fitzgen created PR review comment:

Instead of bailing out when an engine doesn't support a config, can we have a pre-pass where the engine is given mutable access to the config to turn off anything that it doesn't support? This just seems like better use of fuzzing time than rejecting iterations and bailing out.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2022 at 16:43):

fitzgen submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2022 at 16:43):

fitzgen created PR review comment:

Slightly surprised we would ever have a case where we can't run the test case in Wasmtime. What is even the point at that time?

view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2022 at 16:43):

fitzgen submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2022 at 16:43):

fitzgen submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2022 at 18:43):

jameysharp submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2022 at 18:43):

jameysharp created PR review comment:

I think the issue this is addressing is if testing Wasmtime against Wasmtime has been disabled using the ALLOWED_ENGINES environment variable.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2022 at 19:15):

alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2022 at 19:15):

alexcrichton created PR review comment:

Yeah I've come around to feeling this way as well, I'll work on refactoring to enable this

view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2022 at 19:16):

alexcrichton created PR review comment:

Ah yeah this is only applicable when you're doing something like ALLOWED_ENGINES=wasmi locally where I was trying to only differentially-execute against wasmi

view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2022 at 19:16):

alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2022 at 19:16):

alexcrichton merged PR #4845.


Last updated: Dec 23 2024 at 12:05 UTC