Stream: git-wasmtime

Topic: wasmtime / issue #3298 Add a new API to make `Func::call`...

view this post on Zulip Wasmtime GitHub notifications bot (Sep 03 2021 at 18:34):

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

Subscribe to Label Action

cc @peterhuene

<details>
This issue or pull request has been labeled: "wasmtime:api", "wasmtime:c-api"

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 (Sep 08 2021 at 15:28):

alexcrichton commented on issue #3298:

Pondering this more, I'm not happy with this. I'm going to try the alternate strategies to speed up Func::call and see what happens.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 08 2021 at 21:28):

alexcrichton commented on issue #3298:

Ok I have an alternative implementation with less (not zero, just less) API impact at https://github.com/bytecodealliance/wasmtime/pull/3319. I'm currently favoring that one despite it being slightly slower in benchmarks because there's still so much movement of Val values, but I'm not sure that this new API added here is really worth it.

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

alexcrichton commented on issue #3298:

I'm going to close this because I don't think that this is the best way to optimize this, it futzes with too many APIs and will be annoying to maintain I think.

Last updated: Nov 22 2024 at 16:03 UTC