Stream: git-wasmtime

Topic: wasmtime / PR #3293 Remove an indirect function call in `...


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

alexcrichton opened PR #3293 from less-dynamic to main:

This commit optimizes the runtime execution of Func::new by removing
an indirect function call that happens whenever a host function is
called. This indirection was generally done to prevent monomoprhizing a
lot into consumer code but the few extra functions this makes
monomorphic are fairly small, and in general wasm->host call performance
is pretty important.

While not a massive win this is expected to improve codegen, especially
because with the indirect call removed the compiler should now be able
to prove more often when a Func::new closure doesn't panic or return
an error.

<!--

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 (Sep 03 2021 at 16:46):

bjorn3 submitted PR review.

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

bjorn3 created PR review comment:

I think it would be a bit cleaner to inline this function given how small it is.

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

cfallin submitted PR review.

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

alexcrichton updated PR #3293 from less-dynamic to main.

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

abrown submitted PR review.

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

abrown created PR review comment:

You probably meant to remove this?

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

alexcrichton updated PR #3293 from less-dynamic to main.

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

alexcrichton submitted PR review.

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

alexcrichton created PR review comment:

Oops, yes!

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

alexcrichton merged PR #3293.


Last updated: Dec 23 2024 at 12:05 UTC