Separate decoding module peak memory bytes for asm/wasm.

Separate decoding module peak memory bytes for asm/wasm.

Currently, V8 uses the same counter to measue module decoding peak memory bytes
for both asm.js and WASM. This CL separates the counter into two separate
counters, and then uses the appropriate counter when decoding a module.

BUG=chromium:704922
R=bbudge@chromium.org,bradnelson@chromium.org,mtrofin@chromium.org

Review-Url: https://codereview.chromium.org/2778973003
Cr-Commit-Position: refs/heads/master@{#44207}
Committed: https://chromium.googlesource.com/v8/v8/+/759db9fcc88b3c99f4365f2db7d4441f284e76c1

[kschimpf, 2017-03-28 15:15:36]

Description was changed from

==========
Separate decoding module peak memory bytes for asm/wasm.

Currently, V8 uses the same counter to measue module decoding peak memory bytes
for both asm.js and WASM. This CL separates the counter into two separate
counters, and then uses the appropriate counter when decoding a module.

BUG=chromium:704922
==========

to

==========
Separate decoding module peak memory bytes for asm/wasm.

Currently, V8 uses the same counter to measue module decoding peak memory bytes
for both asm.js and WASM. This CL separates the counter into two separate
counters, and then uses the appropriate counter when decoding a module.

BUG=chromium:704922
R=bbudge@chromium.org,bradnelson@chromium.org
==========

[kschimpf, 2017-03-28 15:15:36]

kschimpf@chromium.org changed reviewers:
+ bbudge@chromium.org, bradnelson@chromium.org

[kschimpf, 2017-03-28 15:15:54]

The CQ bit was checked by kschimpf@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-28 15:15:59]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-28 15:42:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-28 15:42:54]

Dry run: This issue passed the CQ dry run.

[bbudge, 2017-03-28 17:14:31]

lgtm

[kschimpf, 2017-03-28 18:13:52]

Description was changed from

==========
Separate decoding module peak memory bytes for asm/wasm.

Currently, V8 uses the same counter to measue module decoding peak memory bytes
for both asm.js and WASM. This CL separates the counter into two separate
counters, and then uses the appropriate counter when decoding a module.

BUG=chromium:704922
R=bbudge@chromium.org,bradnelson@chromium.org
==========

to

==========
Separate decoding module peak memory bytes for asm/wasm.

Currently, V8 uses the same counter to measue module decoding peak memory bytes
for both asm.js and WASM. This CL separates the counter into two separate
counters, and then uses the appropriate counter when decoding a module.

BUG=chromium:704922
R=bbudge@chromium.org,bradnelson@chromium.org,mtrofin@chromium.org
==========

[kschimpf, 2017-03-28 18:13:52]

kschimpf@chromium.org changed reviewers:
+ mtrofin@chromium.org

[kschimpf, 2017-03-28 18:14:21]

Need approval from either mtrofin or bradnelson to commit.

[bradnelson, 2017-03-28 18:14:27]

The CQ bit was checked by bradnelson@chromium.org

[bradnelson, 2017-03-28 18:14:28]

lgtm

[commit-bot: I haz the power, 2017-03-28 18:14:34]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-28 18:20:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-28 18:21:00]

Try jobs failed on following builders:
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/37879)

[kschimpf, 2017-03-28 19:24:43]

The CQ bit was checked by kschimpf@chromium.org

[kschimpf, 2017-03-28 19:24:43]

The patchset sent to the CQ was uploaded after l-g-t-m from bbudge@chromium.org,
bradnelson@chromium.org
Link to the patchset: https://codereview.chromium.org/2778973003/#ps20001
(title: "Fix merge with master conflicts.")

[commit-bot: I haz the power, 2017-03-28 19:24:51]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-28 20:00:11]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490729083433650,
"parent_rev": "c95ef4af70d8c02b48774058141e706ea445d59c", "commit_rev":
"759db9fcc88b3c99f4365f2db7d4441f284e76c1"}

[commit-bot: I haz the power, 2017-03-28 20:00:16]

Description was changed from

==========
Separate decoding module peak memory bytes for asm/wasm.

Currently, V8 uses the same counter to measue module decoding peak memory bytes
for both asm.js and WASM. This CL separates the counter into two separate
counters, and then uses the appropriate counter when decoding a module.

BUG=chromium:704922
R=bbudge@chromium.org,bradnelson@chromium.org,mtrofin@chromium.org
==========

to

==========
Separate decoding module peak memory bytes for asm/wasm.

Currently, V8 uses the same counter to measue module decoding peak memory bytes
for both asm.js and WASM. This CL separates the counter into two separate
counters, and then uses the appropriate counter when decoding a module.

BUG=chromium:704922
R=bbudge@chromium.org,bradnelson@chromium.org,mtrofin@chromium.org

Review-Url: https://codereview.chromium.org/2778973003
Cr-Commit-Position: refs/heads/master@{#44207}
Committed:
https://chromium.googlesource.com/v8/v8/+/759db9fcc88b3c99f4365f2db7d4441f284...
==========

[commit-bot: I haz the power, 2017-03-28 20:00:18]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/v8/v8/+/759db9fcc88b3c99f4365f2db7d4441f284...