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

Unified Diff: pkg/compiler/lib/src/enqueue.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
« no previous file with comments | « pkg/compiler/lib/src/elements/types.dart ('k') | pkg/compiler/lib/src/js_backend/backend_usage.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/compiler/lib/src/enqueue.dart
diff --git a/pkg/compiler/lib/src/enqueue.dart b/pkg/compiler/lib/src/enqueue.dart
index e9a44818ab66452597081b5df7d72a5dbb08b945..fcc594c86760ee137f7cdb8c9af6e0576701739c 100644
--- a/pkg/compiler/lib/src/enqueue.dart
+++ b/pkg/compiler/lib/src/enqueue.dart
@@ -278,7 +278,8 @@ class ResolutionEnqueuer extends EnqueuerImpl {
_registerClosurizedMember(member);
}
if (useSet.contains(MemberUse.CLOSURIZE_STATIC)) {
- applyImpact(listener.registerGetOfStaticFunction());
+ applyImpact(listener.registerGetOfStaticFunction(),
+ impactSource: 'get of static function');
Johnni Winther 2017/04/20 13:27:39 These are added to help debugging. We current only
}
}
@@ -290,10 +291,12 @@ class ResolutionEnqueuer extends EnqueuerImpl {
// We only tell the backend once that [cls] was instantiated, so
// any additional dependencies must be treated as global
// dependencies.
- applyImpact(listener.registerInstantiatedClass(cls));
+ applyImpact(listener.registerInstantiatedClass(cls),
+ impactSource: 'instantiated class');
}
if (useSet.contains(ClassUse.IMPLEMENTED)) {
- applyImpact(listener.registerImplementedClass(cls));
+ applyImpact(listener.registerImplementedClass(cls),
+ impactSource: 'implemented class');
}
}
@@ -306,7 +309,8 @@ class ResolutionEnqueuer extends EnqueuerImpl {
void processConstantUse(ConstantUse constantUse) {
task.measure(() {
if (_worldBuilder.registerConstantUse(constantUse)) {
- applyImpact(listener.registerUsedConstant(constantUse.value));
+ applyImpact(listener.registerUsedConstant(constantUse.value),
+ impactSource: 'constant use');
_recentConstants = true;
}
});
@@ -372,7 +376,8 @@ class ResolutionEnqueuer extends EnqueuerImpl {
void _registerClosurizedMember(MemberEntity element) {
assert(element.isInstanceMember);
- applyImpact(listener.registerClosurizedMember(element));
+ applyImpact(listener.registerClosurizedMember(element),
+ impactSource: 'closurized member');
_worldBuilder.registerClosurizedMember(element);
}
@@ -437,7 +442,8 @@ class ResolutionEnqueuer extends EnqueuerImpl {
entity, "Resolution work list is closed. Trying to add $entity.");
}
- applyImpact(listener.registerUsedElement(entity));
+ applyImpact(listener.registerUsedElement(entity),
+ impactSource: 'used element');
_worldBuilder.registerUsedElement(entity);
_queue.add(workItem);
}
« no previous file with comments | « pkg/compiler/lib/src/elements/types.dart ('k') | pkg/compiler/lib/src/js_backend/backend_usage.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698