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

Issue 8463027: Optimize boolean asserts (Closed)

Created:
9 years, 1 month ago by Jennifer Messerly
Modified:
9 years, 1 month ago
Reviewers:
jimhug
CC:
reviews_dartlang.org, Bob Nystrom
Visibility:
Public.

Description

Optimize boolean asserts This change adds a notion of non-nullability to the type system. We use this to track nullable vs non-nullable bools, removing a lot of the checks in "if" "for" etc. Also fixes ?: precedence, uncovered by sra's PEG tests Committed: https://code.google.com/p/dart/source/detail?r=1481

Patch Set 1 #

Patch Set 2 : co19 status #

Total comments: 5
Unified diffs Side-by-side diffs Delta from patch set Stats (+1582 lines, -1383 lines) Patch
M frog/corejs.dart View 1 chunk +3 lines, -2 lines 0 comments Download
M frog/frogsh View 601 chunks +1434 lines, -1316 lines 1 comment Download
M frog/gen.dart View 4 chunks +9 lines, -7 lines 0 comments Download
M frog/member.dart View 10 chunks +38 lines, -17 lines 0 comments Download
M frog/parser.dart View 2 chunks +4 lines, -2 lines 0 comments Download
M frog/type.dart View 3 chunks +66 lines, -2 lines 3 comments Download
M frog/value.dart View 6 chunks +16 lines, -31 lines 1 comment Download
M frog/world.dart View 2 chunks +4 lines, -0 lines 0 comments Download
M tests/co19/co19-frog.status View 1 4 chunks +8 lines, -6 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Jennifer Messerly
tbr
9 years, 1 month ago (2011-11-11 23:32:12 UTC) #1
jimhug
lgtm http://codereview.chromium.org/8463027/diff/2001/frog/frogsh File frog/frogsh (right): http://codereview.chromium.org/8463027/diff/2001/frog/frogsh#newcode268 frog/frogsh:268: i < this._arguments.length; i++) { I love the ...
9 years, 1 month ago (2011-11-12 00:26:36 UTC) #2
Jennifer Messerly
9 years, 1 month ago (2011-11-12 00:33:00 UTC) #3
http://codereview.chromium.org/8463027/diff/2001/frog/type.dart
File frog/type.dart (right):

http://codereview.chromium.org/8463027/diff/2001/frog/type.dart#newcode366
frog/type.dart:366: class NonNullableType extends Type {
On 2011/11/12 00:26:36, jimhug wrote:
> This seems like a decent choice - but it shares the ugliness of ConcreteType
in
> terms of having to forward way too much.  I'm not sure but I suspect there's a
> better design here that we'll get to over time.

Funny huh? Nothing obviously better sprung to mind, as much as I wanted to
design it better.

I think I lose my privileges to complain about ConcreteType now :)

Powered by Google App Engine
This is Rietveld 408576698