Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(541)

Issue 2556963003: [wasm] record stats at compile time (Closed)

Created:
4 years ago by mtrofin
Modified:
4 years ago
CC:
v8-reviews_googlegroups.com
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

[wasm] record stats at compile time We were losing the stats for the first instance. Recording them as soon as code objects are produced. This way, we have them available for compile-only benchmarks. Committed: https://crrev.com/00803fe0ef6c94b1ea3cd3d7b8d55aa219440a65 Cr-Commit-Position: refs/heads/master@{#41574}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -0 lines) Patch
M src/wasm/wasm-module.cc View 2 chunks +2 lines, -0 lines 0 comments Download

Messages

Total messages: 16 (11 generated)
Mircea Trofin
4 years ago (2016-12-07 21:24:45 UTC) #6
bradnelson
lgtm
4 years ago (2016-12-08 08:37:37 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2556963003/1
4 years ago (2016-12-08 08:37:51 UTC) #11
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years ago (2016-12-08 08:39:30 UTC) #14
commit-bot: I haz the power
4 years ago (2016-12-08 08:39:54 UTC) #16
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/00803fe0ef6c94b1ea3cd3d7b8d55aa219440a65
Cr-Commit-Position: refs/heads/master@{#41574}

Powered by Google App Engine
This is Rietveld 408576698