Commit c7375b20 authored by Clemens Backes's avatar Clemens Backes Committed by Commit Bot

[wasm][serialization] Parallelize copy and relocate

The copy&relocate work is fully parallelizable. Even though it's not the
bottleneck, there is no reason to restrict it to a single thread.

R=thibaudm@chromium.org

Bug: v8:11164
Change-Id: I7d0fea66b2a14e36cb60438cff84e4722b7a7993
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2644949
Commit-Queue: Clemens Backes <clemensb@chromium.org>
Reviewed-by: 's avatarThibaud Michaud <thibaudm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#72366}
parent dc18dce6
......@@ -576,11 +576,8 @@ class CopyAndRelocTask : public JobTask {
} while (!delegate->ShouldYield());
}
size_t GetMaxConcurrency(size_t worker_count) const override {
// Run the CopyAndRelocTask in a single thread for now. We can later bump
// this to see if it improves performance.
if (worker_count > 0) return 0;
return std::min(size_t{1}, from_queue_->NumBatches());
size_t GetMaxConcurrency(size_t /* worker_count */) const override {
return from_queue_->NumBatches();
}
private:
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment