Stream: git-wasmtime

Topic: wasmtime / issue #7083 `wasmtime::CoreDump::_serialize` s...

Wasmtime GitHub notifications bot (Sep 25 2023 at 17:05):

fitzgen opened issue #7083:

Follow up to https://github.com/bytecodealliance/wasmtime/pull/7078#discussion_r1336021323

It is there because the method calls public APIs that require a store with a T but none of them actually use the T. For every pub fn foo<T> that is called, we could create a pub(crate) fn _foo without the T and then call those methods from _serialize and ultimately remove the T from _serialize.

This would make it so that calling wasmtime::CoreDump::serialize isn't monomorphized for every T in a store passed to it, which can help compile times when there is more than one T used in the program.

Last updated: Nov 22 2024 at 16:03 UTC