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

Side by Side Diff: pkg/compiler/lib/src/ssa/graph_builder.dart

Issue 2926863002: Handle parameters in compile_from_dill_test (Closed)
Patch Set: Fixes Created 3 years, 6 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
OLDNEW
1 // Copyright (c) 2016, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2016, 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 import '../closure.dart' show ClosureClassMaps; 5 import '../closure.dart' show ClosureClassMaps;
6 import '../constants/constant_system.dart'; 6 import '../constants/constant_system.dart';
7 import '../common/codegen.dart' show CodegenRegistry; 7 import '../common/codegen.dart' show CodegenRegistry;
8 import '../common_elements.dart'; 8 import '../common_elements.dart';
9 import '../compiler.dart'; 9 import '../compiler.dart';
10 import '../deferred_load.dart'; 10 import '../deferred_load.dart';
(...skipping 147 matching lines...) Expand 10 before | Expand all | Expand 10 after
158 158
159 /// Indicates whether the current block is dead (because it has a throw or a 159 /// Indicates whether the current block is dead (because it has a throw or a
160 /// return further up). If this is false, then [current] may be null. If the 160 /// return further up). If this is false, then [current] may be null. If the
161 /// block is dead then it may also be aborted, but for simplicity we only 161 /// block is dead then it may also be aborted, but for simplicity we only
162 /// abort on statement boundaries, not in the middle of expressions. See 162 /// abort on statement boundaries, not in the middle of expressions. See
163 /// [isAborted]. 163 /// [isAborted].
164 bool isReachable = true; 164 bool isReachable = true;
165 165
166 HParameterValue lastAddedParameter; 166 HParameterValue lastAddedParameter;
167 167
168 Map<ParameterElement, HInstruction> parameters = 168 Map<Local, HInstruction> parameters = <Local, HInstruction>{};
169 <ParameterElement, HInstruction>{};
170 169
171 HBasicBlock addNewBlock() { 170 HBasicBlock addNewBlock() {
172 HBasicBlock block = graph.addNewBlock(); 171 HBasicBlock block = graph.addNewBlock();
173 // If adding a new block during building of an expression, it is due to 172 // If adding a new block during building of an expression, it is due to
174 // conditional expressions or short-circuit logical operators. 173 // conditional expressions or short-circuit logical operators.
175 return block; 174 return block;
176 } 175 }
177 176
178 void open(HBasicBlock block) { 177 void open(HBasicBlock block) {
179 block.open(); 178 block.open();
(...skipping 117 matching lines...) Expand 10 before | Expand all | Expand 10 after
297 return mirrorsData.mustRetainMetadata; 296 return mirrorsData.mustRetainMetadata;
298 case 'USE_CONTENT_SECURITY_POLICY': 297 case 'USE_CONTENT_SECURITY_POLICY':
299 return options.useContentSecurityPolicy; 298 return options.useContentSecurityPolicy;
300 case 'IS_FULL_EMITTER': 299 case 'IS_FULL_EMITTER':
301 return !USE_LAZY_EMITTER && !options.useStartupEmitter; 300 return !USE_LAZY_EMITTER && !options.useStartupEmitter;
302 default: 301 default:
303 return null; 302 return null;
304 } 303 }
305 } 304 }
306 } 305 }
OLDNEW
« no previous file with comments | « pkg/compiler/lib/src/ssa/builder_kernel.dart ('k') | pkg/compiler/lib/src/ssa/kernel_ast_adapter.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698