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

Issue 2707023002: Train ddc snapshot on itself (Closed)

Created:
3 years, 10 months ago by vsm
Modified:
3 years, 10 months ago
Reviewers:
Jacob
CC:
reviews_dartlang.org, siva, Bill Hesse
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Train ddc snapshot on itself This speeds up dartdevc on hello world from about 600ms to about 150ms. OTOH, it also increases the snapshot size from ~4.6M to ~31M (the latter is just under the size of the dart2js snapshot). R=jacobr@google.com Committed: https://github.com/dart-lang/sdk/commit/e4733ac1457225b98c1b78f10b883ebad24fc81c

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -1 line) Patch
M utils/dartdevc/BUILD.gn View 1 chunk +6 lines, -1 line 1 comment Download

Messages

Total messages: 7 (3 generated)
vsm
3 years, 10 months ago (2017-02-21 14:41:17 UTC) #3
Jacob
lgtm https://codereview.chromium.org/2707023002/diff/1/utils/dartdevc/BUILD.gn File utils/dartdevc/BUILD.gn (right): https://codereview.chromium.org/2707023002/diff/1/utils/dartdevc/BUILD.gn#newcode16 utils/dartdevc/BUILD.gn:16: "--unsafe-force-compile", random idea: can we train it on ...
3 years, 10 months ago (2017-02-21 15:28:52 UTC) #4
vsm
On 2017/02/21 15:28:52, Jacob wrote: > lgtm > > https://codereview.chromium.org/2707023002/diff/1/utils/dartdevc/BUILD.gn > File utils/dartdevc/BUILD.gn (right): > ...
3 years, 10 months ago (2017-02-21 15:36:33 UTC) #5
vsm
3 years, 10 months ago (2017-02-21 23:56:26 UTC) #7
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
e4733ac1457225b98c1b78f10b883ebad24fc81c (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698