Commit e242540e authored by Clemens Hammacher's avatar Clemens Hammacher Committed by Commit Bot

[wasm] Fill max mem pages counter with data

The counter is dead since January 2018 (https://crrev.com/c/860011).
This CL reenables it.

R=titzer@chromium.org

Bug: v8:8975
Change-Id: I0245edd40bb972a89e223685f56ab02fbdf68aa0
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1514677Reviewed-by: 's avatarBen Titzer <titzer@chromium.org>
Commit-Queue: Clemens Hammacher <clemensh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#60166}
parent 7f77cd9f
......@@ -258,6 +258,12 @@ MaybeHandle<WasmInstanceObject> InstanceBuilder::Build() {
auto initial_pages_counter = SELECT_WASM_COUNTER(
isolate_->counters(), module_->origin, wasm, min_mem_pages_count);
initial_pages_counter->AddSample(initial_pages);
if (module_->has_maximum_pages) {
DCHECK_EQ(kWasmOrigin, module_->origin);
auto max_pages_counter =
isolate_->counters()->wasm_wasm_max_mem_pages_count();
max_pages_counter->AddSample(module_->maximum_pages);
}
// Asm.js has memory_ already set at this point, so we don't want to
// overwrite it.
if (memory_.is_null()) {
......
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