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

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

Issue 2916353002: Support locals in compile_from_dill_test. (Closed)
Patch Set: 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 'package:kernel/ast.dart' as ir; 5 import 'package:kernel/ast.dart' as ir;
6 6
7 import '../closure.dart'; 7 import '../closure.dart';
8 import '../common.dart'; 8 import '../common.dart';
9 import '../common/codegen.dart' show CodegenRegistry; 9 import '../common/codegen.dart' show CodegenRegistry;
10 import '../common/names.dart'; 10 import '../common/names.dart';
(...skipping 2099 matching lines...) Expand 10 before | Expand all | Expand 10 after
2110 HInstruction initialValue = pop(); 2110 HInstruction initialValue = pop();
2111 2111
2112 _visitLocalSetter(declaration, initialValue); 2112 _visitLocalSetter(declaration, initialValue);
2113 2113
2114 // Ignore value 2114 // Ignore value
2115 pop(); 2115 pop();
2116 } 2116 }
2117 } 2117 }
2118 2118
2119 void _visitLocalSetter(ir.VariableDeclaration variable, HInstruction value) { 2119 void _visitLocalSetter(ir.VariableDeclaration variable, HInstruction value) {
2120 LocalElement local = _localsMap.getLocal(variable); 2120 Local local = _localsMap.getLocal(variable);
2121 2121
2122 // Give the value a name if it doesn't have one already. 2122 // Give the value a name if it doesn't have one already.
2123 if (value.sourceElement == null) { 2123 if (value.sourceElement == null) {
2124 value.sourceElement = local; 2124 value.sourceElement = local;
2125 } 2125 }
2126 2126
2127 stack.add(value); 2127 stack.add(value);
2128 localsHandler.updateLocal( 2128 localsHandler.updateLocal(
2129 local, 2129 local,
2130 typeBuilder.potentiallyCheckOrTrustType( 2130 typeBuilder.potentiallyCheckOrTrustType(
(...skipping 1275 matching lines...) Expand 10 before | Expand all | Expand 10 after
3406 enterBlock.setBlockFlow( 3406 enterBlock.setBlockFlow(
3407 new HTryBlockInformation( 3407 new HTryBlockInformation(
3408 kernelBuilder.wrapStatementGraph(bodyGraph), 3408 kernelBuilder.wrapStatementGraph(bodyGraph),
3409 exception, 3409 exception,
3410 kernelBuilder.wrapStatementGraph(catchGraph), 3410 kernelBuilder.wrapStatementGraph(catchGraph),
3411 kernelBuilder.wrapStatementGraph(finallyGraph)), 3411 kernelBuilder.wrapStatementGraph(finallyGraph)),
3412 exitBlock); 3412 exitBlock);
3413 kernelBuilder.inTryStatement = previouslyInTryStatement; 3413 kernelBuilder.inTryStatement = previouslyInTryStatement;
3414 } 3414 }
3415 } 3415 }
OLDNEW
« no previous file with comments | « pkg/compiler/lib/src/kernel/kernel_backend_strategy.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