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

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

Issue 2595493002: Merge CoreTypes and CoreClasses into CommonElements. (Closed)
Patch Set: Updated cf. comment Created 4 years 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
« no previous file with comments | « pkg/compiler/lib/src/compile_time_constants.dart ('k') | pkg/compiler/lib/src/constant_system_dart.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/compiler/lib/src/compiler.dart
diff --git a/pkg/compiler/lib/src/compiler.dart b/pkg/compiler/lib/src/compiler.dart
index 026d5e926e0c8fd3ae58be2a49c8a9738c4669ec..71639ec9df71645ef9fd22855ae90369bf6c2f18 100644
--- a/pkg/compiler/lib/src/compiler.dart
+++ b/pkg/compiler/lib/src/compiler.dart
@@ -10,7 +10,6 @@ import '../compiler_new.dart' as api;
import 'cache_strategy.dart' show CacheStrategy;
import 'closure.dart' as closureMapping show ClosureTask;
import 'common/backend_api.dart' show Backend;
-import 'common/codegen.dart' show CodegenWorkItem;
import 'common/names.dart' show Selectors;
import 'common/names.dart' show Identifiers, Uris;
import 'common/resolution.dart'
@@ -25,7 +24,7 @@ import 'common/work.dart' show WorkItem;
import 'common.dart';
import 'compile_time_constants.dart';
import 'constants/values.dart';
-import 'core_types.dart' show CoreClasses, CommonElements, CoreTypes;
+import 'core_types.dart' show CommonElements;
import 'dart_types.dart' show DartType, DynamicType, InterfaceType, Types;
import 'deferred_load.dart' show DeferredLoadTask;
import 'diagnostics/code_location.dart';
@@ -90,7 +89,7 @@ abstract class Compiler implements LibraryLoaderListener {
final IdGenerator idGenerator = new IdGenerator();
Types types;
- _CompilerCoreTypes _coreTypes;
+ _CompilerCommonElements _commonElements;
CompilerDiagnosticReporter _reporter;
CompilerResolution _resolution;
ParsingContext _parsingContext;
@@ -133,9 +132,7 @@ abstract class Compiler implements LibraryLoaderListener {
FunctionElement mainFunction;
DiagnosticReporter get reporter => _reporter;
- CommonElements get commonElements => _coreTypes;
- CoreClasses get coreClasses => _coreTypes;
- CoreTypes get coreTypes => _coreTypes;
+ CommonElements get commonElements => _commonElements;
Resolution get resolution => _resolution;
ParsingContext get parsingContext => _parsingContext;
@@ -210,9 +207,7 @@ abstract class Compiler implements LibraryLoaderListener {
_reporter = new CompilerDiagnosticReporter(this, options);
}
_resolution = createResolution();
- // TODO(johnniwinther): Initialize core types in [initializeCoreClasses] and
- // make its field final.
- _coreTypes = new _CompilerCoreTypes(_resolution, reporter);
+ _commonElements = new _CompilerCommonElements(_resolution, reporter);
types = new Types(_resolution);
if (options.verbose) {
@@ -346,7 +341,7 @@ abstract class Compiler implements LibraryLoaderListener {
/// Note that [library] has not been scanned yet, nor has its imports/exports
/// been resolved.
void onLibraryCreated(LibraryElement library) {
- _coreTypes.onLibraryCreated(library);
+ _commonElements.onLibraryCreated(library);
backend.onLibraryCreated(library);
}
@@ -1127,7 +1122,7 @@ class SuppressionInfo {
int hints = 0;
}
-class _CompilerCoreTypes implements CoreTypes, CoreClasses, CommonElements {
+class _CompilerCommonElements implements CommonElements {
final Resolution resolution;
final DiagnosticReporter reporter;
@@ -1144,7 +1139,7 @@ class _CompilerCoreTypes implements CoreTypes, CoreClasses, CommonElements {
// specific backend.
LibraryElement jsHelperLibrary;
- _CompilerCoreTypes(this.resolution, this.reporter);
+ _CompilerCommonElements(this.resolution, this.reporter);
// From dart:core
@@ -1914,12 +1909,6 @@ class CompilerResolution implements Resolution {
ParsingContext get parsingContext => _compiler.parsingContext;
@override
- CoreClasses get coreClasses => _compiler.coreClasses;
-
- @override
- CoreTypes get coreTypes => _compiler.coreTypes;
-
- @override
CommonElements get commonElements => _compiler.commonElements;
@override
@@ -1948,7 +1937,7 @@ class CompilerResolution implements Resolution {
_compiler.mirrorUsageAnalyzerTask;
@override
- LibraryElement get coreLibrary => _compiler._coreTypes.coreLibrary;
+ LibraryElement get coreLibrary => _compiler._commonElements.coreLibrary;
@override
bool get wasProxyConstantComputedTestingOnly => _proxyConstant != null;
« no previous file with comments | « pkg/compiler/lib/src/compile_time_constants.dart ('k') | pkg/compiler/lib/src/constant_system_dart.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698