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

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: Comments 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 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
129 } 130 }
130 131
131 Statement visitSetField(SetField node) { 132 Statement visitSetField(SetField node) {
132 visitExpression(node.object); 133 visitExpression(node.object);
133 visitExpression(node.value); 134 visitExpression(node.value);
134 node.next = visitStatement(node.next); 135 node.next = visitStatement(node.next);
135 return node; 136 return node;
136 } 137 }
137 138
138 } 139 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698