Stream: wasmtime

Topic: heads-up: API change for Module::new()


view this post on Zulip Yury Delendik (May 27 2020 at 20:01):

So https://github.com/bytecodealliance/wasmtime/pull/1761 also proposes API change on what we store in Module. It drastically changes how Module::newand Instance::new works: now Instance needs directly hold Storeand Module don't. On bright side, we can send Module across the threads \m/

To make wasmtime more threads friendly, this PR: Moves CodeMemory, VMInterrupts and SignatureRegistry from Compiler CompiledModule holds CodeMemory and GdbJitImageRegistration Store keeps track of...

view this post on Zulip Dan Gohman (May 27 2020 at 20:07):

While here, can we remove the module field from Instance?

view this post on Zulip Yury Delendik (May 27 2020 at 21:10):

We will be there after this PR, perhaps :)

view this post on Zulip Yury Delendik (May 27 2020 at 21:17):

@Dan Gohman yeah, this change will not drive API change. I think removing module requires better CodeMemory ownership management than we have now.


Last updated: Nov 22 2024 at 16:03 UTC