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

Issue 721093003: dart2js: simplify ssa-generate constant.

Created:
6 years, 1 month ago by floitsch
Modified:
6 years, 1 month ago
Reviewers:
Johnni Winther, sigurdm
CC:
reviews_dartlang.org, sigurdm
Visibility:
Public.

Description

dart2js: simplify ssa-generate constant. The registry was already doing the work that was in generateConstant. R=sigurdm@google.com Committed: https://code.google.com/p/dart/source/detail?r=41715 Reverted: https://code.google.com/p/dart/source/detail?r=41720

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -19 lines) Patch
M pkg/compiler/lib/src/ssa/codegen.dart View 1 chunk +1 line, -19 lines 1 comment Download

Messages

Total messages: 7 (2 generated)
floitsch
6 years, 1 month ago (2014-11-13 14:39:16 UTC) #2
sigurdm
LGTM
6 years, 1 month ago (2014-11-13 14:40:01 UTC) #3
floitsch
Committed patchset #1 (id:1) manually as 41715 (presubmit successful).
6 years, 1 month ago (2014-11-13 14:40:19 UTC) #4
floitsch
Had to revert because of unused functions: == Hint: The method 'registerTypeConstant' is never called. ...
6 years, 1 month ago (2014-11-13 16:47:48 UTC) #6
Johnni Winther
6 years, 1 month ago (2014-11-14 08:33:32 UTC) #7
https://codereview.chromium.org/721093003/diff/1/pkg/compiler/lib/src/ssa/cod...
File pkg/compiler/lib/src/ssa/codegen.dart (left):

https://codereview.chromium.org/721093003/diff/1/pkg/compiler/lib/src/ssa/cod...
pkg/compiler/lib/src/ssa/codegen.dart:1821: if (constant.isType) {
We need to move this to JavaScriptBackend.registerCompileTimeConstantInternal
under a flag that the constant occurred in executed code (i.e. not in metadata).
It is needed to detect registration of custom elements.

Powered by Google App Engine
This is Rietveld 408576698