• Aseem Garg's avatar
    [wasm] Reland "[wasm] redirect wasm calls to js functions through a GCed table" · 649b1e70
    Aseem Garg authored
    This reverts commit 25f03308.
    
    Reason for revert: Fix the cause of bot failure and reland
    
    Original change's description:
    > Revert "[wasm] redirect wasm calls to js functions through a GCed table"
    > 
    > This reverts commit eb65f35e.
    > 
    > Reason for revert: Broke jetstream benchmark on android.
    > 
    > BUG=chromium:750828
    > 
    > Original change's description:
    > > [wasm] redirect wasm calls to js functions through a GCed table
    > > 
    > > With this patch, rather than embedding the JSReceiver address directly
    > > in the WasmToJS wrappers, we put that in a fixed array with global handle
    > > scope and instead embed the location of the handle and the index in the
    > > wrapper. This ensures that the wrapper doesn't need to be patched if the
    > > GC kicks in. This is needed to get the WASM code off the GCed heap.
    > > 
    > > R=​mtrofin@chromium.org
    > > 
    > > Bug: 
    > > Change-Id: Ie5a77a78cdecec51b04f702c63b8e4285e6a2d8d
    > > Reviewed-on: https://chromium-review.googlesource.com/581682
    > > Commit-Queue: Aseem Garg <aseemgarg@chromium.org>
    > > Reviewed-by: Mircea Trofin <mtrofin@chromium.org>
    > > Cr-Commit-Position: refs/heads/master@{#46884}
    > 
    > TBR=mtrofin@chromium.org,aseemgarg@google.com,aseemgarg@chromium.org,clemensh@chromium.org
    > 
    > # Not skipping CQ checks because original CL landed > 1 day ago.
    > 
    > Change-Id: I26f49ee0a1fe73cc5d8852ded87b56638be39ebf
    > Reviewed-on: https://chromium-review.googlesource.com/596268
    > Commit-Queue: Aseem Garg <aseemgarg@chromium.org>
    > Reviewed-by: Clemens Hammacher <clemensh@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#47059}
    
    R=mtrofin@chromium.org,aseemgarg@google.com,aseemgarg@chromium.org,clemensh@chromium.org,sullivan@chromium.org
    
    Change-Id: I29ef35f6e612a706d9f571da3e7beb1da8b5052b
    Bug: chromium:750828
    Reviewed-on: https://chromium-review.googlesource.com/597010
    Commit-Queue: Aseem Garg <aseemgarg@chromium.org>
    Reviewed-by: 's avatarClemens Hammacher <clemensh@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#47177}
    649b1e70
wasm-module.cc 46.3 KB