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

Issue 1080003004: Optimize identical based on types. (Closed)

Created:
5 years, 8 months ago by karlklose
Modified:
5 years, 6 months ago
Reviewers:
asgerf
CC:
reviews_dartlang.org, Kevin Millikin (Google)
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : Rebase and update. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+42 lines, -12 lines) Patch
M pkg/compiler/lib/src/cps_ir/type_propagation.dart View 1 6 chunks +21 lines, -6 lines 0 comments Download
M tests/compiler/dart2js/js_backend_cps_ir_control_flow_test.dart View 1 1 chunk +2 lines, -1 line 0 comments Download
M tests/compiler/dart2js/js_backend_cps_ir_operators_test.dart View 2 chunks +19 lines, -4 lines 0 comments Download
M tests/standalone/standalone.status View 1 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 7 (0 generated)
karlklose
5 years, 8 months ago (2015-04-17 10:39:17 UTC) #1
asgerf
How expensive is TypeMask.intersection? It looks like an optimization that rarely clicks in practice, so ...
5 years, 8 months ago (2015-04-20 14:23:17 UTC) #2
asgerf
In the interest of getting this resolved I ran some benchmarks and could not measure ...
5 years, 7 months ago (2015-05-19 15:15:07 UTC) #3
karlklose
On 2015/05/19 15:15:07, asgerf wrote: > In the interest of getting this resolved I ran ...
5 years, 7 months ago (2015-05-20 09:15:25 UTC) #4
karlklose
Rebased and added missing check for nullability.
5 years, 6 months ago (2015-06-01 12:38:59 UTC) #5
asgerf
On 2015/06/01 12:38:59, karlklose wrote: > Rebased and added missing check for nullability. No patch ...
5 years, 6 months ago (2015-06-01 12:39:59 UTC) #6
karlklose
5 years, 6 months ago (2015-06-02 11:01:48 UTC) #7
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
add53e33e773a0e1445a32b4b4289519ce745abf (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698