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

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=177305

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

Cool!

> 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>