• Annie Sullivan's avatar
    Revert "[wasm] redirect wasm calls to js functions through a GCed table" · 25f03308
    Annie Sullivan authored
    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: 's avatarClemens Hammacher <clemensh@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#47059}
    25f03308
wasm-interpreter.cc 89 KB