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

Side by Side Diff: pkg/compiler/lib/src/tree_ir/optimization/loop_rewriter.dart

Issue 958603002: Added VariableUse expression to tree IR. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Updated docs regarding catch parameters Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 part of tree_ir.optimization; 5 part of tree_ir.optimization;
6 6
7 /// Rewrites [WhileTrue] statements with an [If] body into a [WhileCondition], 7 /// Rewrites [WhileTrue] statements with an [If] body into a [WhileCondition],
8 /// in situations where only one of the branches contains a [Continue] to the 8 /// in situations where only one of the branches contains a [Continue] to the
9 /// loop. Schematically: 9 /// loop. Schematically:
10 /// 10 ///
(...skipping 25 matching lines...) Expand all
36 } 36 }
37 37
38 Statement visitLabeledStatement(LabeledStatement node) { 38 Statement visitLabeledStatement(LabeledStatement node) {
39 node.body = visitStatement(node.body); 39 node.body = visitStatement(node.body);
40 node.next = visitStatement(node.next); 40 node.next = visitStatement(node.next);
41 return node; 41 return node;
42 } 42 }
43 43
44 Statement visitAssign(Assign node) { 44 Statement visitAssign(Assign node) {
45 // Clean up redundant assignments left behind in the previous phase. 45 // Clean up redundant assignments left behind in the previous phase.
46 if (node.variable == node.definition) { 46 Expression def = node.definition;
47 if (def is VariableUse && node.variable == def.variable) {
47 --node.variable.readCount; 48 --node.variable.readCount;
48 --node.variable.writeCount; 49 --node.variable.writeCount;
49 return visitStatement(node.next); 50 return visitStatement(node.next);
50 } 51 }
51 visitExpression(node.definition); 52 visitExpression(node.definition);
52 node.next = visitStatement(node.next); 53 node.next = visitStatement(node.next);
53 return node; 54 return node;
54 } 55 }
55 56
56 Statement visitReturn(Return node) { 57 Statement visitReturn(Return node) {
(...skipping 78 matching lines...) Expand 10 before | Expand all | Expand 10 after
135 } 136 }
136 137
137 Statement visitSetField(SetField node) { 138 Statement visitSetField(SetField node) {
138 visitExpression(node.object); 139 visitExpression(node.object);
139 visitExpression(node.value); 140 visitExpression(node.value);
140 node.next = visitStatement(node.next); 141 node.next = visitStatement(node.next);
141 return node; 142 return node;
142 } 143 }
143 144
144 } 145 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698