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

Issue 2649513003: [turbolizer] distinguish optimization attempts (Closed)

Created:
3 years, 11 months ago by Tobias Tebbi
Modified:
3 years, 11 months ago
Reviewers:
danno
CC:
v8-reviews_googlegroups.com
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

[turbolizer] distinguish optimization attempts Create different json files for diffferent optimization attempts of the same function. This allows to analyze the graphs of early optimization attempts that lead to deopts or to produce a proper json file in the case of a deopt loop. R=danno@chromium.org Review-Url: https://codereview.chromium.org/2649513003 Cr-Commit-Position: refs/heads/master@{#42567} Committed: https://chromium.googlesource.com/v8/v8/+/dc93bb47400bc1079639926b6234aea174acd692

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -2 lines) Patch
M src/compiler/graph-visualizer.cc View 1 chunk +8 lines, -2 lines 0 comments Download

Messages

Total messages: 7 (3 generated)
Tobias Tebbi
3 years, 11 months ago (2017-01-20 14:29:57 UTC) #1
danno
Nice! lgtm.
3 years, 11 months ago (2017-01-20 14:53:51 UTC) #2
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/2649513003/1
3 years, 11 months ago (2017-01-20 14:55:02 UTC) #4
commit-bot: I haz the power
3 years, 11 months ago (2017-01-20 15:30:10 UTC) #7
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/dc93bb47400bc1079639926b6234aea174a...

Powered by Google App Engine
This is Rietveld 408576698