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

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

Issue 2827283002: Implement subtype relation for kernel based elements. (Closed)
Patch Set: Add remaining checks of BackendUsage Created 3 years, 8 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
Index: pkg/compiler/lib/src/js_backend/resolution_listener.dart
diff --git a/pkg/compiler/lib/src/js_backend/resolution_listener.dart b/pkg/compiler/lib/src/js_backend/resolution_listener.dart
index e1d07ae1e4c1dbc377d10835b1511244700f7142..b93283823b73dd771aefd0894784b5f7908bb1f4 100644
--- a/pkg/compiler/lib/src/js_backend/resolution_listener.dart
+++ b/pkg/compiler/lib/src/js_backend/resolution_listener.dart
@@ -181,15 +181,18 @@ class ResolutionEnqueuerListener extends EnqueuerListener {
void onQueueOpen(Enqueuer enqueuer, FunctionEntity mainMethod,
Iterable<LibraryEntity> libraries) {
if (_deferredLoadTask.isProgramSplit) {
- enqueuer.applyImpact(_computeDeferredLoadingImpact());
+ enqueuer.applyImpact(_computeDeferredLoadingImpact(),
+ impactSource: 'deferred load');
}
- enqueuer.applyImpact(_nativeEnqueuer.processNativeClasses(libraries));
+ enqueuer.applyImpact(_nativeEnqueuer.processNativeClasses(libraries),
+ impactSource: 'native classes');
if (mainMethod != null) {
- enqueuer.applyImpact(_computeMainImpact(mainMethod));
+ enqueuer.applyImpact(_computeMainImpact(mainMethod),
+ impactSource: 'main impact');
}
// Elements required by enqueueHelpers are global dependencies
// that are not pulled in by a particular element.
- enqueuer.applyImpact(computeHelpersImpact());
+ enqueuer.applyImpact(computeHelpersImpact(), impactSource: 'helpers');
}
@override
@@ -199,9 +202,12 @@ class ResolutionEnqueuerListener extends EnqueuerListener {
//
// Return early if any elements are added to avoid counting the elements as
// due to mirrors.
- enqueuer.applyImpact(_customElementsAnalysis.flush());
- enqueuer.applyImpact(_lookupMapResolutionAnalysis.flush());
- enqueuer.applyImpact(_typeVariableResolutionAnalysis.flush());
+ enqueuer.applyImpact(_customElementsAnalysis.flush(),
+ impactSource: _customElementsAnalysis);
+ enqueuer.applyImpact(_lookupMapResolutionAnalysis.flush(),
+ impactSource: _lookupMapResolutionAnalysis);
+ enqueuer.applyImpact(_typeVariableResolutionAnalysis.flush(),
+ impactSource: _typeVariableResolutionAnalysis);
for (ClassEntity cls in recentClasses) {
MemberEntity element =

Powered by Google App Engine
This is Rietveld 408576698