[Webkit-unassigned] [Bug 170690] WebAssembly: report Memory usage to GC

[bugzilla-daemon at webkit.org]

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

JF Bastien <jfbastien at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
           Assignee|webkit-unassigned at lists.web |jfbastien at apple.com
                   |kit.org                     |
 Attachment #309699|                            |review?, commit-queue?
              Flags|                            |

--- Comment #2 from JF Bastien <jfbastien at apple.com> ---
Created attachment 309699

  --> https://bugs.webkit.org/attachment.cgi?id=309699&action=review

patch

I'm still not sure this is the right thing. We definitely should tell the GC about this memory but:

 1. It's more memory than other allocations. Maybe that's the same as ArrayBuffer and doesn't matter?
 2. We're not sure all pages are dirty, so we could be over-reporting if the developer over-allocates. We just don't know if they do.

I think these don't matter, and we can adjust if they do, so this patch is probably right. I'd nonetheless would rather ask and get feedback on these points.

-- 
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/20170511/47a439eb/attachment.html>