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

Unified Diff: pkg/compiler/lib/src/resolution/tree_elements.dart

Issue 2468303003: dart2js: move type-mask intermediate data from tree-elements into the global (Closed)
Patch Set: Created 4 years, 1 month 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 side-by-side diff with in-line comments
Download patch
Index: pkg/compiler/lib/src/resolution/tree_elements.dart
diff --git a/pkg/compiler/lib/src/resolution/tree_elements.dart b/pkg/compiler/lib/src/resolution/tree_elements.dart
index bae0857e364b3618983f79db46880fc08a06a544..fd0080fd231836a9e938866e56a001d2a5804a62 100644
--- a/pkg/compiler/lib/src/resolution/tree_elements.dart
+++ b/pkg/compiler/lib/src/resolution/tree_elements.dart
@@ -10,7 +10,6 @@ import '../dart_types.dart';
import '../diagnostics/source_span.dart';
import '../elements/elements.dart';
import '../tree/tree.dart';
-import '../types/types.dart' show TypeMask;
Johnni Winther 2016/11/03 08:50:21 Nice!!
import '../universe/selector.dart' show Selector;
import '../util/util.dart';
import 'secret_tree_element.dart' show getTreeElement, setTreeElement;
@@ -36,21 +35,9 @@ abstract class TreeElements {
Selector getGetterSelectorInComplexSendSet(SendSet node);
Selector getOperatorSelectorInComplexSendSet(SendSet node);
DartType getType(Node node);
- TypeMask getTypeMask(Node node);
- TypeMask getGetterTypeMaskInComplexSendSet(SendSet node);
- TypeMask getOperatorTypeMaskInComplexSendSet(SendSet node);
- void setTypeMask(Node node, TypeMask mask);
- void setGetterTypeMaskInComplexSendSet(SendSet node, TypeMask mask);
- void setOperatorTypeMaskInComplexSendSet(SendSet node, TypeMask mask);
/// Returns the for-in loop variable for [node].
Element getForInVariable(ForIn node);
- TypeMask getIteratorTypeMask(ForIn node);
- TypeMask getMoveNextTypeMask(ForIn node);
- TypeMask getCurrentTypeMask(ForIn node);
- void setIteratorTypeMask(ForIn node, TypeMask mask);
- void setMoveNextTypeMask(ForIn node, TypeMask mask);
- void setCurrentTypeMask(ForIn node, TypeMask mask);
void setConstant(Node node, ConstantExpression constant);
ConstantExpression getConstant(Node node);
@@ -107,7 +94,6 @@ abstract class TreeElements {
class TreeElementMapping extends TreeElements {
final AnalyzableElement analyzedElement;
Map<Spannable, Selector> _selectors;
- Map<Spannable, TypeMask> _typeMasks;
Map<Node, DartType> _types;
Map<Node, DartType> _typesCache;
@@ -442,79 +428,6 @@ class TreeElementMapping extends TreeElements {
return _targetLabels != null ? _targetLabels[node] : null;
}
- TypeMask _getTypeMask(Spannable node) {
- return _typeMasks != null ? _typeMasks[node] : null;
- }
-
- void _setTypeMask(Spannable node, TypeMask mask) {
- if (_typeMasks == null) {
- _typeMasks = new Maplet<Spannable, TypeMask>();
- }
- _typeMasks[node] = mask;
- }
-
- @override
- void setTypeMask(Node node, TypeMask mask) {
- _setTypeMask(node, mask);
- }
-
- @override
- TypeMask getTypeMask(Node node) => _getTypeMask(node);
-
- @override
- void setGetterTypeMaskInComplexSendSet(SendSet node, TypeMask mask) {
- _setTypeMask(node.selector, mask);
- }
-
- @override
- TypeMask getGetterTypeMaskInComplexSendSet(SendSet node) {
- return _getTypeMask(node.selector);
- }
-
- @override
- void setOperatorTypeMaskInComplexSendSet(SendSet node, TypeMask mask) {
- _setTypeMask(node.assignmentOperator, mask);
- }
-
- @override
- TypeMask getOperatorTypeMaskInComplexSendSet(SendSet node) {
- return _getTypeMask(node.assignmentOperator);
- }
-
- // The following methods set selectors on the "for in" node. Since
- // we're using three selectors, we need to use children of the node,
- // and we arbitrarily choose which ones.
-
- @override
- void setIteratorTypeMask(ForIn node, TypeMask mask) {
- _setTypeMask(node, mask);
- }
-
- @override
- TypeMask getIteratorTypeMask(ForIn node) {
- return _getTypeMask(node);
- }
-
- @override
- void setMoveNextTypeMask(ForIn node, TypeMask mask) {
- _setTypeMask(node.forToken, mask);
- }
-
- @override
- TypeMask getMoveNextTypeMask(ForIn node) {
- return _getTypeMask(node.forToken);
- }
-
- @override
- void setCurrentTypeMask(ForIn node, TypeMask mask) {
- _setTypeMask(node.inToken, mask);
- }
-
- @override
- TypeMask getCurrentTypeMask(ForIn node) {
- return _getTypeMask(node.inToken);
- }
-
void registerNativeData(Node node, dynamic nativeData) {
if (_nativeData == null) {
_nativeData = <Node, dynamic>{};

Powered by Google App Engine
This is Rietveld 408576698