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

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

Issue 2782503003: Extract NativeClassResolver from NativeResolutionEnqueuer (Closed)
Patch Set: Remove unused getter. Created 3 years, 9 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | pkg/compiler/lib/src/native/enqueue.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/compiler/lib/src/js_backend/backend.dart
diff --git a/pkg/compiler/lib/src/js_backend/backend.dart b/pkg/compiler/lib/src/js_backend/backend.dart
index 03c5e0fc16375c82108a3704faae3dd3e6a6a15d..8887796936989efe9556415e9aef2b5d6e642c97 100644
--- a/pkg/compiler/lib/src/js_backend/backend.dart
+++ b/pkg/compiler/lib/src/js_backend/backend.dart
@@ -511,9 +511,6 @@ class JavaScriptBackend {
_checkedModeHelpers = new CheckedModeHelpers(commonElements, helpers);
emitter =
new CodeEmitterTask(compiler, generateSourceMap, useStartupEmitter);
- _nativeResolutionEnqueuer = new native.NativeResolutionEnqueuer(compiler);
- _nativeCodegenEnqueuer = new native.NativeCodegenEnqueuer(
- compiler, emitter, _nativeResolutionEnqueuer);
_typeVariableResolutionAnalysis = new TypeVariableResolutionAnalysis(
compiler.elementEnvironment, impacts, backendUsageBuilder);
@@ -817,7 +814,7 @@ class JavaScriptBackend {
}
mirrorsDataBuilder.computeMembersNeededForReflection(
compiler.enqueuer.resolution.worldBuilder, closedWorld);
- _backendUsage = _backendUsageBuilder.close();
+ _backendUsage = backendUsageBuilder.close();
_rtiNeed = rtiNeedBuilder.computeRuntimeTypesNeed(
compiler.enqueuer.resolution.worldBuilder,
closedWorld,
@@ -826,8 +823,7 @@ class JavaScriptBackend {
helpers,
_backendUsage,
enableTypeAssertions: compiler.options.enableTypeAssertions);
- _interceptorData =
- _interceptorDataBuilder.onResolutionComplete(closedWorld);
+ _interceptorData = interceptorDataBuilder.onResolutionComplete(closedWorld);
_oneShotInterceptorData =
new OneShotInterceptorData(interceptorData, helpers);
mirrorsResolutionAnalysis.onResolutionComplete();
@@ -890,6 +886,10 @@ class JavaScriptBackend {
CompilerTask task, Compiler compiler) {
_nativeBasicData =
nativeBasicDataBuilder.close(compiler.elementEnvironment);
+ _nativeResolutionEnqueuer = new native.NativeResolutionEnqueuer(
+ compiler,
+ new NativeClassResolverImpl(
+ compiler.resolution, reporter, helpers, nativeBasicData));
_nativeData = new NativeDataImpl(nativeBasicData);
_backendClasses = new JavaScriptBackendClasses(
compiler.elementEnvironment, helpers, nativeBasicData);
@@ -935,16 +935,16 @@ class JavaScriptBackend {
impacts,
backendClasses,
nativeBasicData,
- _interceptorDataBuilder,
- _backendUsageBuilder,
- _rtiNeedBuilder,
+ interceptorDataBuilder,
+ backendUsageBuilder,
+ rtiNeedBuilder,
mirrorsDataBuilder,
noSuchMethodRegistry,
customElementsResolutionAnalysis,
lookupMapResolutionAnalysis,
mirrorsResolutionAnalysis,
typeVariableResolutionAnalysis,
- _nativeResolutionEnqueuer),
+ nativeResolutionEnqueuer),
new ElementResolutionWorldBuilder(
this, compiler.resolution, const OpenWorldStrategy()),
new ResolutionWorkItemBuilder(compiler.resolution));
@@ -972,6 +972,8 @@ class JavaScriptBackend {
backendClasses,
helpers,
nativeBasicData);
+ _nativeCodegenEnqueuer = new native.NativeCodegenEnqueuer(
+ compiler, emitter, _nativeResolutionEnqueuer);
return new CodegenEnqueuer(
task,
compiler.options,
@@ -991,7 +993,7 @@ class JavaScriptBackend {
typeVariableCodegenAnalysis,
lookupMapAnalysis,
mirrorsCodegenAnalysis,
- _nativeCodegenEnqueuer));
+ nativeCodegenEnqueuer));
}
WorldImpact codegen(CodegenWorkItem work) {
« no previous file with comments | « no previous file | pkg/compiler/lib/src/native/enqueue.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698