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

Issue 952973004: dart2js: create constants lazily in the new emitter. (Closed)

Created:
5 years, 10 months ago by floitsch
Modified:
5 years, 9 months ago
Reviewers:
zarah
CC:
reviews_dartlang.org, herhut
Target Ref:
refs/remotes/git-svn
Visibility:
Public.

Description

dart2js: create constants lazily in the new emitter. R=zarah@google.com Committed: https://code.google.com/p/dart/source/detail?r=44326

Patch Set 1 #

Patch Set 2 : Don't print setup time. #

Patch Set 3 : Rebase #

Patch Set 4 : Make the constant object non-polymorphic. #

Patch Set 5 : Rebase. #

Patch Set 6 : Minor fixes. #

Patch Set 7 : Special case literal-expressions in the unparser. #

Patch Set 8 : Remove unnecessary parenthesis. #

Patch Set 9 : Add constants to deferred libraries. #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+60 lines, -20 lines) Patch
M pkg/compiler/lib/src/js_emitter/new_emitter/model_emitter.dart View 1 2 3 4 5 6 7 8 10 chunks +60 lines, -20 lines 2 comments Download

Messages

Total messages: 8 (1 generated)
floitsch
5 years, 10 months ago (2015-02-24 23:28:52 UTC) #2
floitsch
Made it such that the constants aren't making the C object polymorphic. (Huge performance win). ...
5 years, 10 months ago (2015-02-25 17:06:24 UTC) #3
zarah
LGTM
5 years, 10 months ago (2015-02-26 08:01:50 UTC) #4
floitsch
PTAL. I fixed a few more constants in the native_helper and remove the bad "constants" ...
5 years, 9 months ago (2015-02-27 12:42:23 UTC) #5
zarah
Still lgtm.
5 years, 9 months ago (2015-03-06 09:15:29 UTC) #6
floitsch
Made minor changes. Will commit on Monday. Note: this patch uncovered a latent bug in ...
5 years, 9 months ago (2015-03-06 19:30:26 UTC) #7
floitsch
5 years, 9 months ago (2015-03-09 16:37:12 UTC) #8
Message was sent while issue was closed.
Committed patchset #9 (id:160001) manually as 44326 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698