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

Issue 11035009: Enable inlining of control flow, increase code heap size 12 -> 18Mb (note that code will be moved t… (Closed)

Created:
8 years, 2 months ago by srdjan
Modified:
8 years, 2 months ago
Reviewers:
siva
CC:
reviews_dartlang.org, Kevin Millikin (Google), zerny-google
Visibility:
Public.

Description

Enable inlining of control flow, increase code heap size 12 -> 18Mb (note that code will be moved to old space where it can grow). Print heap size at exit of VM using trace-isolates. Code heap increase because of inline-control-flow (dart2js): production mode: 4,143k -> 5,143k checked mode: 10,789k -> 14,545k Committed: https://code.google.com/p/dart/source/detail?r=13085

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -2 lines) Patch
M runtime/vm/dart.cc View 1 chunk +3 lines, -0 lines 0 comments Download
M runtime/vm/flow_graph_inliner.cc View 1 chunk +1 line, -1 line 0 comments Download
M runtime/vm/heap.h View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
srdjan
8 years, 2 months ago (2012-10-01 20:02:40 UTC) #1
siva
8 years, 2 months ago (2012-10-01 20:08:48 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698