[Webkit-unassigned] [Bug 177305] WebAssembly: cache memory address / size on instance

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Dec 4 17:05:41 PST 2017


--- Comment #2 from JF Bastien <jfbastien at apple.com> ---
(In reply to GSkachkov from comment #1)
> I can try to fix this issue.


> Wasm::Instance will have to add property cachedMemory and cachedSize,

Yes, and use it in the JIT instead of doing the indirection.

> Wasm::Memory should contain links to Wasm::Instance where Wasm::Memory is
> used to iterate within `grow`, is it correct understating of the issue?

Wasm::Memory can be used by multiple Wasm::Instance at the same time, so the Wasm::Memory has to know all of the Wasm::Instances that refer to it. The Instance owns the memory, so the back-pointers need to be weak references. Then yes, you would update all of them from grow.

You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20171205/9c7bb18b/attachment.html>

More information about the webkit-unassigned mailing list