[wasm][asm.js] Add wasm module size to asm debug measurements.

[wasm][asm.js] Add wasm module size to asm debug measurements.

Also show size when --trace-asm-time is on.

BUG=
R=mtrofin@chromium.org

Review-Url: https://codereview.chromium.org/2658703003
Cr-Commit-Position: refs/heads/master@{#42678}
Committed: https://chromium.googlesource.com/v8/v8/+/f6d0dc6d889f70890a8ee6f00091d14f0ef6199a

[bradnelson, 2017-01-26 07:17:42]

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

[commit-bot: I haz the power, 2017-01-26 07:17:52]

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

[bradn, 2017-01-26 07:17:57]

bradnelson@google.com changed reviewers:
+ bradnelson@google.com

[Mircea Trofin, 2017-01-26 07:19:24]

On 2017/01/26 07:17:58, bradn wrote:

lgtm, 1 nit

[Mircea Trofin, 2017-01-26 07:19:31]

https://codereview.chromium.org/2658703003/diff/1/src/asmjs/asm-js.cc
File src/asmjs/asm-js.cc (right):

https://codereview.chromium.org/2658703003/diff/1/src/asmjs/asm-js.cc#newcode194
src/asmjs/asm-js.cc:194: uintptr_t wasm_size = module->end() - module->begin();
DCHECK end >= begin

[bradn, 2017-01-26 07:46:12]

https://codereview.chromium.org/2658703003/diff/1/src/asmjs/asm-js.cc
File src/asmjs/asm-js.cc (right):

https://codereview.chromium.org/2658703003/diff/1/src/asmjs/asm-js.cc#newcode194
src/asmjs/asm-js.cc:194: uintptr_t wasm_size = module->end() - module->begin();
On 2017/01/26 07:19:31, Mircea Trofin wrote:
> DCHECK end >= begin

Done.

[bradn, 2017-01-26 07:46:34]

The CQ bit was checked by bradnelson@google.com

[bradn, 2017-01-26 07:46:35]

The patchset sent to the CQ was uploaded after l-g-t-m from mtrofin@chromium.org
Link to the patchset: https://codereview.chromium.org/2658703003/#ps20001
(title: "fix")

[commit-bot: I haz the power, 2017-01-26 07:46:44]

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-26 08:17:07]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1485416794605600,
"parent_rev": "d0befa956f13b279a6e8dfdeb19eda8e5a14e9ca", "commit_rev":
"f6d0dc6d889f70890a8ee6f00091d14f0ef6199a"}

[commit-bot: I haz the power, 2017-01-26 08:17:12]

Description was changed from

==========
[wasm][asm.js] Add wasm module size to asm debug measurements.

Also show size when --trace-asm-time is on.

BUG=
R=mtrofin@chromium.org
==========

to

==========
[wasm][asm.js] Add wasm module size to asm debug measurements.

Also show size when --trace-asm-time is on.

BUG=
R=mtrofin@chromium.org

Review-Url: https://codereview.chromium.org/2658703003
Cr-Commit-Position: refs/heads/master@{#42678}
Committed:
https://chromium.googlesource.com/v8/v8/+/f6d0dc6d889f70890a8ee6f00091d14f0ef...
==========

[commit-bot: I haz the power, 2017-01-26 08:17:12]

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