[wasm] Add timer to measure asm->wasm translation

[wasm] Add timer to measure asm->wasm translation

This will be used for perf tests:
https://chromereviews.googleplex.com/565327014/

R=titzer@chromium.org, bradnelson@chromium.org

Review-Url: https://codereview.chromium.org/2663713003
Cr-Commit-Position: refs/heads/master@{#42807}
Committed: https://chromium.googlesource.com/v8/v8/+/8bc60063ef596b27cfb60be40370d4d94ed97392

[Clemens Hammacher, 2017-01-30 18:48:33]

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

[commit-bot: I haz the power, 2017-01-30 18:48:45]

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

[Clemens Hammacher, 2017-01-30 18:54:38]

Description was changed from

==========
[wasm] Add timer to measure asm->wasm translation

This will be used for perf tests.

R=titzer@chromium.org
==========

to

==========
[wasm] Add timer to measure asm->wasm translation

This will be used for perf tests:
https://chromereviews.googleplex.com/565327014/

R=titzer@chromium.org, bradnelson@chromium.org
==========

[Clemens Hammacher, 2017-01-30 18:54:39]

clemensh@chromium.org changed reviewers:
+ bradnelson@chromium.org

[titzer, 2017-01-30 19:02:56]

On 2017/01/30 18:54:48, Clemens Hammacher wrote:

lgtm

[commit-bot: I haz the power, 2017-01-30 19:23:13]

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

[commit-bot: I haz the power, 2017-01-30 19:23:14]

Dry run: This issue passed the CQ dry run.

[bradn, 2017-01-31 02:46:59]

bradnelson@google.com changed reviewers:
+ bradnelson@google.com, isherman@chromium.org

[bradn, 2017-01-31 02:47:00]

LGTM, however, by the way, none of these metrics is hooked up on the chrome side
(for actual collection).

I got told by the metrics folks they'd like to review the size of this histogram
buckets before we hook them up.

Also, I've been meaning to check if the ones we have are still intact (some
predate parallel compilation and might not be in the right places in light of
that).

[bradn, 2017-01-31 02:47:17]

+isherman to look at bucket sizes.

[Clemens Hammacher, 2017-01-31 08:31:11]

The CQ bit was checked by clemensh@chromium.org

[commit-bot: I haz the power, 2017-01-31 08:31:19]

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-01-31 08:34:36]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485851471925250, "parent_rev":
"b8df95499338c86cf29bb8502a488a5ebf41c415", "commit_rev":
"8bc60063ef596b27cfb60be40370d4d94ed97392"}

[commit-bot: I haz the power, 2017-01-31 08:34:41]

Description was changed from

==========
[wasm] Add timer to measure asm->wasm translation

This will be used for perf tests:
https://chromereviews.googleplex.com/565327014/

R=titzer@chromium.org, bradnelson@chromium.org
==========

to

==========
[wasm] Add timer to measure asm->wasm translation

This will be used for perf tests:
https://chromereviews.googleplex.com/565327014/

R=titzer@chromium.org, bradnelson@chromium.org

Review-Url: https://codereview.chromium.org/2663713003
Cr-Commit-Position: refs/heads/master@{#42807}
Committed:
https://chromium.googlesource.com/v8/v8/+/8bc60063ef596b27cfb60be40370d4d94ed...
==========

[commit-bot: I haz the power, 2017-01-31 08:34:42]

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

[Ilya Sherman, 2017-02-01 01:08:43]

On 2017/01/31 02:47:17, bradn wrote:
> +isherman to look at bucket sizes.

Sorry, I'm not quite following -- what are the bucket sizes?  And, who mentioned
that they would like to review the sizes?  (I don't think it was me
specifically, though I do agree that a sanity-check can be helpful.)

[bradn, 2017-02-02 00:03:12]

Sorry misrecalled the state I think.
Sending you a CL for the other half (or rather reviving an old one).

[bradn, 2017-02-02 00:03:13]

Sorry misrecalled the state I think.
Sending you a CL for the other half (or rather reviving an old one).

[bradn, 2017-02-02 00:11:11]

Added this to the chrome side CL:
https://codereview.chromium.org/1890773002/