Stream: git-wasmtime

Topic: wasmtime / issue #3156 Remove rss prediction from api_cal...


view this post on Zulip Wasmtime GitHub notifications bot (Aug 06 2021 at 14:51):

github-actions[bot] commented on issue #3156:

Subscribe to Label Action

cc @fitzgen

<details>
This issue or pull request has been labeled: "fuzzing"

Thus the following users have been cc'd because of the following labels:

To subscribe or unsubscribe from this label, edit the <code>.github/subscribe-to-label.json</code> configuration file.

Learn more.
</details>

view this post on Zulip Wasmtime GitHub notifications bot (Aug 06 2021 at 17:43):

alexcrichton commented on issue #3156:

Personally I think the fuzz target is still good to have. It's the only fuzz target that can exercise a variable number of modules/instances within the store (although they still aren't linked to one another). I do think that its usefulness has decreased over time but not to the point that I think it should go away.

It's also theoretically a great place to add more API calls in the future too!

view this post on Zulip Wasmtime GitHub notifications bot (Aug 06 2021 at 18:05):

fitzgen commented on issue #3156:

Well module linking exercises multiple modules/instances in the same store now. But sure.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 06 2021 at 18:27):

alexcrichton commented on issue #3156:

Oh, true! In that case I think you're right in that this may have outlived its usefulness a bit. I don't mind either way though.


Last updated: Dec 23 2024 at 12:05 UTC