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

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

Issue 2786593004: Use entities in NativeResolutionEnqueuer and make it reusable for kernel (Closed)
Patch Set: 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 | « pkg/compiler/lib/src/compiler.dart ('k') | pkg/compiler/lib/src/kernel/native_class_resolver.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 600f23c5b169572aa7b8fa71d6c6d7dd4b7ac16a..625abbda4c0747a1e571d4e6794fc8f003ea3ec9 100644
--- a/pkg/compiler/lib/src/js_backend/backend.dart
+++ b/pkg/compiler/lib/src/js_backend/backend.dart
@@ -843,13 +843,18 @@ class JavaScriptBackend {
CompilerTask task, Compiler compiler) {
_nativeBasicData =
nativeBasicDataBuilder.close(compiler.elementEnvironment);
+ _backendClasses = new JavaScriptBackendClasses(
+ compiler.elementEnvironment, helpers, nativeBasicData);
_nativeResolutionEnqueuer = new native.NativeResolutionEnqueuer(
- compiler,
+ compiler.options,
+ compiler.elementEnvironment,
+ commonElements,
+ helpers,
+ backendClasses,
+ backendUsageBuilder,
new NativeClassResolverImpl(
compiler.resolution, reporter, helpers, nativeBasicData));
_nativeData = new NativeDataImpl(nativeBasicData);
- _backendClasses = new JavaScriptBackendClasses(
- compiler.elementEnvironment, helpers, nativeBasicData);
_customElementsResolutionAnalysis = new CustomElementsResolutionAnalysis(
compiler.resolution,
constantSystem,
@@ -865,7 +870,7 @@ class JavaScriptBackend {
commonElements,
impacts,
nativeBasicData,
- nativeResolutionEnqueuer,
+ _nativeResolutionEnqueuer,
backendUsageBuilder,
mirrorsDataBuilder,
customElementsResolutionAnalysis,
@@ -896,7 +901,7 @@ class JavaScriptBackend {
lookupMapResolutionAnalysis,
mirrorsResolutionAnalysis,
typeVariableResolutionAnalysis,
- nativeResolutionEnqueuer,
+ _nativeResolutionEnqueuer,
kernelTask),
new ElementResolutionWorldBuilder(
this, compiler.resolution, const OpenWorldStrategy()),
@@ -926,7 +931,14 @@ class JavaScriptBackend {
helpers,
nativeBasicData);
_nativeCodegenEnqueuer = new native.NativeCodegenEnqueuer(
- compiler, emitter, _nativeResolutionEnqueuer);
+ compiler.options,
+ compiler.elementEnvironment,
+ commonElements,
+ helpers,
+ backendClasses,
+ emitter,
+ _nativeResolutionEnqueuer,
+ nativeData);
return new CodegenEnqueuer(
task,
compiler.options,
@@ -1019,6 +1031,8 @@ class JavaScriptBackend {
return worldImpact;
}
+ // TODO(johnniwinther): Remove this. It is now only used for testing.
+ @deprecated
native.NativeEnqueuer get nativeResolutionEnqueuer =>
_nativeResolutionEnqueuer;
« no previous file with comments | « pkg/compiler/lib/src/compiler.dart ('k') | pkg/compiler/lib/src/kernel/native_class_resolver.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698