• 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
Name
Last commit
Last update
..
benchmarks Loading commit data...
cctest Loading commit data...
common Loading commit data...
debugger Loading commit data...
fuzzer Loading commit data...
inspector Loading commit data...
intl Loading commit data...
js-perf-test Loading commit data...
memory Loading commit data...
message Loading commit data...
mjsunit Loading commit data...
mkgrokdump Loading commit data...
mozilla Loading commit data...
preparser Loading commit data...
promises-aplus Loading commit data...
test262 Loading commit data...
unittests Loading commit data...
wasm-spec-tests Loading commit data...
webkit Loading commit data...
BUILD.gn Loading commit data...
bot_default.gyp Loading commit data...
bot_default.isolate Loading commit data...
default.gyp Loading commit data...
default.isolate Loading commit data...
optimize_for_size.gyp Loading commit data...
optimize_for_size.isolate Loading commit data...
perf.gyp Loading commit data...
perf.isolate Loading commit data...