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

Side by Side Diff: pkg/compiler/lib/src/js_backend/codegen/codegen.dart

Issue 1182913003: Split TypedSelector into Selector and TypeMask. (Closed) Base URL: https://github.com/dart-lang/sdk.git@master
Patch Set: Updated cf. comments. Created 5 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) 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 library code_generator; 5 library code_generator;
6 6
7 import 'glue.dart'; 7 import 'glue.dart';
8 8
9 import '../../tree_ir/tree_ir_nodes.dart' as tree_ir; 9 import '../../tree_ir/tree_ir_nodes.dart' as tree_ir;
10 import '../../tree_ir/tree_ir_nodes.dart' show BuiltinOperator; 10 import '../../tree_ir/tree_ir_nodes.dart' show BuiltinOperator;
11 import '../../js/js.dart' as js; 11 import '../../js/js.dart' as js;
12 import '../../elements/elements.dart'; 12 import '../../elements/elements.dart';
13 import '../../io/source_information.dart' show SourceInformation; 13 import '../../io/source_information.dart' show SourceInformation;
14 import '../../util/maplet.dart'; 14 import '../../util/maplet.dart';
15 import '../../constants/values.dart'; 15 import '../../constants/values.dart';
16 import '../../dart2jslib.dart'; 16 import '../../dart2jslib.dart';
17 import '../../dart_types.dart'; 17 import '../../dart_types.dart';
18 import '../../types/types.dart' show TypeMask;
19 import '../../universe/universe.dart' show UniverseSelector;
18 import '../../closure.dart' show ClosureClassElement; 20 import '../../closure.dart' show ClosureClassElement;
19 21
20 class CodegenBailout { 22 class CodegenBailout {
21 final tree_ir.Node node; 23 final tree_ir.Node node;
22 final String reason; 24 final String reason;
23 CodegenBailout(this.node, this.reason); 25 CodegenBailout(this.node, this.reason);
24 String get message { 26 String get message {
25 return 'bailout${node != null ? " on $node" : ""}: $reason'; 27 return 'bailout${node != null ? " on $node" : ""}: $reason';
26 } 28 }
27 } 29 }
(...skipping 218 matching lines...) Expand 10 before | Expand all | Expand 10 after
246 248
247 registry.registerInstantiatedType(node.type); 249 registry.registerInstantiatedType(node.type);
248 Selector selector = node.selector; 250 Selector selector = node.selector;
249 FunctionElement target = node.target; 251 FunctionElement target = node.target;
250 List<js.Expression> arguments = visitExpressionList(node.arguments); 252 List<js.Expression> arguments = visitExpressionList(node.arguments);
251 return buildStaticInvoke(selector, target, arguments); 253 return buildStaticInvoke(selector, target, arguments);
252 } 254 }
253 255
254 void registerMethodInvoke(tree_ir.InvokeMethod node) { 256 void registerMethodInvoke(tree_ir.InvokeMethod node) {
255 Selector selector = node.selector; 257 Selector selector = node.selector;
258 TypeMask mask = node.mask;
256 if (selector.isGetter) { 259 if (selector.isGetter) {
257 registry.registerDynamicGetter(selector); 260 registry.registerDynamicGetter(new UniverseSelector(selector, mask));
258 } else if (selector.isSetter) { 261 } else if (selector.isSetter) {
259 registry.registerDynamicSetter(selector); 262 registry.registerDynamicSetter(new UniverseSelector(selector, mask));
260 } else { 263 } else {
261 assert(invariant(CURRENT_ELEMENT_SPANNABLE, 264 assert(invariant(CURRENT_ELEMENT_SPANNABLE,
262 selector.isCall || selector.isOperator || 265 selector.isCall || selector.isOperator ||
263 selector.isIndex || selector.isIndexSet, 266 selector.isIndex || selector.isIndexSet,
264 message: 'unexpected kind ${selector.kind}')); 267 message: 'unexpected kind ${selector.kind}'));
265 // TODO(sigurdm): We should find a better place to register the call. 268 // TODO(sigurdm): We should find a better place to register the call.
266 Selector call = new Selector.callClosureFrom(selector); 269 Selector call = new Selector.callClosureFrom(selector);
267 registry.registerDynamicInvocation(call); 270 registry.registerDynamicInvocation(new UniverseSelector(call, null));
268 registry.registerDynamicInvocation(selector); 271 registry.registerDynamicInvocation(new UniverseSelector(selector, mask));
269 } 272 }
270 } 273 }
271 274
272 @override 275 @override
273 js.Expression visitInvokeMethod(tree_ir.InvokeMethod node) { 276 js.Expression visitInvokeMethod(tree_ir.InvokeMethod node) {
274 registerMethodInvoke(node); 277 registerMethodInvoke(node);
275 return js.propertyCall(visitExpression(node.receiver), 278 return js.propertyCall(visitExpression(node.receiver),
276 glue.invocationName(node.selector), 279 glue.invocationName(node.selector),
277 visitExpressionList(node.arguments)); 280 visitExpressionList(node.arguments));
278 } 281 }
(...skipping 465 matching lines...) Expand 10 before | Expand all | Expand 10 after
744 return js.js("typeof # === 'number' && Math.floor(#) === #", args); 747 return js.js("typeof # === 'number' && Math.floor(#) === #", args);
745 } 748 }
746 } 749 }
747 750
748 visitFunctionExpression(tree_ir.FunctionExpression node) { 751 visitFunctionExpression(tree_ir.FunctionExpression node) {
749 // FunctionExpressions are currently unused. 752 // FunctionExpressions are currently unused.
750 // We might need them if we want to emit raw JS nested functions. 753 // We might need them if we want to emit raw JS nested functions.
751 throw 'FunctionExpressions should not be used'; 754 throw 'FunctionExpressions should not be used';
752 } 755 }
753 } 756 }
OLDNEW
« no previous file with comments | « pkg/compiler/lib/src/js_backend/backend.dart ('k') | pkg/compiler/lib/src/js_emitter/class_stub_generator.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698