• Clemens Backes's avatar
    Reland "[wasm] Move write scope out of NativeModule::AddCode" · dd152c48
    Clemens Backes authored
    This is a reland of 6ae18c2d, with
    {CompileWasmCapiCallWrapper} fixed to also contain a
    {CodeSpaceWriteScope}.
    
    Original change's description:
    > [wasm] Move write scope out of NativeModule::AddCode
    >
    > {NativeModule::AddCode} is a central method that should usually be
    > called in batches, where the caller holds a {CodeSpaceWriteScope} for a
    > longer time (over several compilations).
    > This CL moves us closer to that by removing the scope from that central
    > method and instead putting it in callers where it becomes more visible.
    > There are already TODOs to introduce caching or batching to avoid some
    > switching, and one more TODO is added.
    >
    > Drive-by: Remove an unneeded {CodeSpaceMemoryModificationScope}.
    >
    > R=jkummerow@chromium.org
    >
    > Bug: v8:11974
    > Change-Id: Ia13c601abc766e5fca6ca053bf1fc4d647b53ed0
    > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3098186
    > Reviewed-by: Jakob Kummerow <jkummerow@chromium.org>
    > Commit-Queue: Clemens Backes <clemensb@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#76344}
    
    Bug: v8:11974
    Cq-Include-Trybots: luci.v8.try:v8_mac_arm64_dbg_ng
    Cq-Include-Trybots: luci.v8.try:v8_mac_arm64_rel_ng
    Change-Id: I6367bbd9dc52c403513eb1a168aa1f6eb4044ca1
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3129703Reviewed-by: 's avatarJakob Kummerow <jkummerow@chromium.org>
    Commit-Queue: Clemens Backes <clemensb@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#76626}
    dd152c48
Name
Last commit
Last update
..
DIR_METADATA Loading commit data...
OWNERS Loading commit data...
test-backing-store.cc Loading commit data...
test-c-wasm-entry.cc Loading commit data...
test-compilation-cache.cc Loading commit data...
test-gc.cc Loading commit data...
test-grow-memory.cc Loading commit data...
test-jump-table-assembler.cc Loading commit data...
test-liftoff-for-fuzzing.cc Loading commit data...
test-liftoff-inspection.cc Loading commit data...
test-run-wasm-64.cc Loading commit data...
test-run-wasm-asmjs.cc Loading commit data...
test-run-wasm-atomics.cc Loading commit data...
test-run-wasm-atomics64.cc Loading commit data...
test-run-wasm-bulk-memory.cc Loading commit data...
test-run-wasm-exceptions.cc Loading commit data...
test-run-wasm-interpreter.cc Loading commit data...
test-run-wasm-js.cc Loading commit data...
test-run-wasm-memory64.cc Loading commit data...
test-run-wasm-module.cc Loading commit data...
test-run-wasm-relaxed-simd.cc Loading commit data...
test-run-wasm-sign-extension.cc Loading commit data...
test-run-wasm-simd-liftoff.cc Loading commit data...
test-run-wasm-simd.cc Loading commit data...
test-run-wasm-wrappers.cc Loading commit data...
test-run-wasm.cc Loading commit data...
test-streaming-compilation.cc Loading commit data...
test-wasm-breakpoints.cc Loading commit data...
test-wasm-codegen.cc Loading commit data...
test-wasm-import-wrapper-cache.cc Loading commit data...
test-wasm-metrics.cc Loading commit data...
test-wasm-serialization.cc Loading commit data...
test-wasm-shared-engine.cc Loading commit data...
test-wasm-stack.cc Loading commit data...
test-wasm-trap-position.cc Loading commit data...
wasm-atomics-utils.h Loading commit data...
wasm-run-utils.cc Loading commit data...
wasm-run-utils.h Loading commit data...
wasm-simd-utils.cc Loading commit data...
wasm-simd-utils.h Loading commit data...