Split counters for functions per module for asm and wasm.

Split counters for functions per module for asm and wasm.

Currently, V8 uses the same counter to count both wasm and and asm js. This
splits the counters into two separate counters, and then uses the appropriate
counter when instantiating the module.

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

Review-Url: https://codereview.chromium.org/2777073003
Cr-Commit-Position: refs/heads/master@{#44164}
Committed: https://chromium.googlesource.com/v8/v8/+/35701006960e4a6419f2dcb62964cfbb311bcd5a

[kschimpf, 2017-03-27 20:10:40]

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

[commit-bot: I haz the power, 2017-03-27 20:10:43]

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-27 20:10:43]

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

[commit-bot: I haz the power, 2017-03-27 20:10:44]

Dry run: No L-G-T-M from a valid reviewer yet.
CQ run can only be started once the patch has received an L-G-T-M from a full
committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,_not_ a
full super star committer.
Committers are members of the group "project-v8-committers".
Note that this has nothing to do with OWNERS files.

[Karl, 2017-03-27 20:19:46]

The CQ bit was checked by kschimpf@google.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-27 20:19:55]

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

[kschimpf, 2017-03-27 20:29:52]

Description was changed from

==========
Split counters for functions per module for asm and wasm.

Currently, V8 uses the same counter to count both wasm and and asm js. This
splits the counters into two separate counters, and then uses the appropriate
counter when instantiating the module.

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

to

==========
Split counters for functions per module for asm and wasm.

Currently, V8 uses the same counter to count both wasm and and asm js. This
splits the counters into two separate counters, and then uses the appropriate
counter when instantiating the module.

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

[kschimpf, 2017-03-27 20:29:52]

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

[bradnelson, 2017-03-27 20:43:22]

lgtm

[commit-bot: I haz the power, 2017-03-27 20:46:21]

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

[commit-bot: I haz the power, 2017-03-27 20:46:22]

Dry run: This issue passed the CQ dry run.

[kschimpf, 2017-03-27 21:08:22]

The CQ bit was checked by kschimpf@chromium.org

[commit-bot: I haz the power, 2017-03-27 21:08:24]

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-27 21:09:37]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490648902144560, "parent_rev":
"15247047e5b63f41d96a509ff91d438ef66f14f2", "commit_rev":
"35701006960e4a6419f2dcb62964cfbb311bcd5a"}

[commit-bot: I haz the power, 2017-03-27 21:09:44]

Description was changed from

==========
Split counters for functions per module for asm and wasm.

Currently, V8 uses the same counter to count both wasm and and asm js. This
splits the counters into two separate counters, and then uses the appropriate
counter when instantiating the module.

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

to

==========
Split counters for functions per module for asm and wasm.

Currently, V8 uses the same counter to count both wasm and and asm js. This
splits the counters into two separate counters, and then uses the appropriate
counter when instantiating the module.

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

Review-Url: https://codereview.chromium.org/2777073003
Cr-Commit-Position: refs/heads/master@{#44164}
Committed:
https://chromium.googlesource.com/v8/v8/+/35701006960e4a6419f2dcb62964cfbb311...
==========

[commit-bot: I haz the power, 2017-03-27 21:09:45]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/35701006960e4a6419f2dcb62964cfbb311...