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

Issue 1783603009: simplify function coercions -- DDC can generate function types in place (Closed)

Created:
4 years, 9 months ago by Jennifer Messerly
Modified:
4 years, 9 months ago
Reviewers:
Leaf
CC:
dev-compiler+reviews_dartlang.org
Base URL:
git@github.com:dart-lang/dev_compiler.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

simplify function coercions -- DDC can generate function types in place R=leafp@google.com Committed: https://github.com/dart-lang/dev_compiler/commit/d051cbd7cb9de070f7210dc557d76df37900867c

Patch Set 1 #

Total comments: 3

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+41 lines, -423 lines) Patch
M lib/runtime/dart/_internal.js View 3 chunks +2 lines, -12 lines 0 comments Download
M lib/runtime/dart/_isolate_helper.js View 2 chunks +1 line, -2 lines 0 comments Download
M lib/runtime/dart/async.js View 20 chunks +14 lines, -32 lines 0 comments Download
M lib/runtime/dart/convert.js View 4 chunks +3 lines, -6 lines 0 comments Download
M lib/runtime/dart/html.js View 3 chunks +2 lines, -4 lines 0 comments Download
M lib/runtime/dart/html_common.js View 2 chunks +1 line, -2 lines 0 comments Download
M lib/runtime/dart/indexed_db.js View 2 chunks +1 line, -2 lines 0 comments Download
M lib/src/codegen/reify_coercions.dart View 1 5 chunks +9 lines, -357 lines 0 comments Download
M lib/src/utils.dart View 1 chunk +7 lines, -0 lines 0 comments Download
M test/codegen/expect/collection/src/priority_queue.js View 2 chunks +1 line, -6 lines 0 comments Download
M test/codegen/expect/language-all.js View Binary file 0 comments Download

Messages

Total messages: 6 (2 generated)
Jennifer Messerly
Seems like we can just emit them in place. We might want caching, but the ...
4 years, 9 months ago (2016-03-12 00:36:56 UTC) #2
Leaf
lgtm https://codereview.chromium.org/1783603009/diff/1/lib/src/codegen/reify_coercions.dart File lib/src/codegen/reify_coercions.dart (right): https://codereview.chromium.org/1783603009/diff/1/lib/src/codegen/reify_coercions.dart#newcode91 lib/src/codegen/reify_coercions.dart:91: /// possibly recording additional coercions functions and typedefs ...
4 years, 9 months ago (2016-03-12 01:14:05 UTC) #3
Jennifer Messerly
Thanks! https://codereview.chromium.org/1783603009/diff/1/lib/src/codegen/reify_coercions.dart File lib/src/codegen/reify_coercions.dart (right): https://codereview.chromium.org/1783603009/diff/1/lib/src/codegen/reify_coercions.dart#newcode91 lib/src/codegen/reify_coercions.dart:91: /// possibly recording additional coercions functions and typedefs ...
4 years, 9 months ago (2016-03-14 17:37:47 UTC) #4
Jennifer Messerly
4 years, 9 months ago (2016-03-14 17:38:03 UTC) #6
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
d051cbd7cb9de070f7210dc557d76df37900867c (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698