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

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

Issue 2323733002: Compute ResolutionImpact directly from kernel, part 1 of ? (Closed)
Patch Set: Updated cf. comments. Created 4 years, 3 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/js_backend/backend.dart ('k') | pkg/compiler/lib/src/kernel/accessors.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/enqueuer.dart
diff --git a/pkg/compiler/lib/src/js_backend/enqueuer.dart b/pkg/compiler/lib/src/js_backend/enqueuer.dart
index 80f241f29cb9d11eb95ac98a6488424d68b5fa52..29bec96ad57e0accfdd65d36fe6b742e9577c24e 100644
--- a/pkg/compiler/lib/src/js_backend/enqueuer.dart
+++ b/pkg/compiler/lib/src/js_backend/enqueuer.dart
@@ -109,8 +109,7 @@ class CodegenEnqueuer implements Enqueuer {
// Codegen inlines field initializers. It only needs to generate
// code for checked setters.
if (element.isField && element.isInstanceMember) {
- if (!options.enableTypeAssertions ||
- element.enclosingElement.isClosure) {
+ if (!options.enableTypeAssertions || element.enclosingElement.isClosure) {
return;
}
}
@@ -145,8 +144,7 @@ class CodegenEnqueuer implements Enqueuer {
universe.registerTypeInstantiation(type,
isNative: isNative,
byMirrors: mirrorUsage, onImplemented: (ClassElement cls) {
- backend
- .registerImplementedClass(cls, this, globalDependencies);
+ backend.registerImplementedClass(cls, this, globalDependencies);
});
// TODO(johnniwinther): Share this reasoning with [Universe].
if (!cls.isAbstract || isNative || mirrorUsage) {
@@ -271,8 +269,7 @@ class CodegenEnqueuer implements Enqueuer {
// We only tell the backend once that [superclass] was instantiated, so
// any additional dependencies must be treated as global
// dependencies.
- backend.registerInstantiatedClass(
- superclass, this, globalDependencies);
+ backend.registerInstantiatedClass(superclass, this, globalDependencies);
}
ClassElement superclass = cls;
@@ -307,8 +304,7 @@ class CodegenEnqueuer implements Enqueuer {
includedEnclosing: enclosingWasIncluded)) {
logEnqueueReflectiveAction(ctor);
ClassElement cls = ctor.declaration.enclosingClass;
- backend.registerInstantiatedType(
- cls.rawType, this, mirrorDependencies,
+ backend.registerInstantiatedType(cls.rawType, this, mirrorDependencies,
mirrorUsage: true);
registerStaticUse(new StaticUse.foreignUse(ctor.declaration));
}
@@ -356,8 +352,7 @@ class CodegenEnqueuer implements Enqueuer {
if (includeClass) {
logEnqueueReflectiveAction(cls, "register");
ClassElement decl = cls.declaration;
- backend.registerInstantiatedType(
- decl.rawType, this, mirrorDependencies,
+ backend.registerInstantiatedType(decl.rawType, this, mirrorDependencies,
mirrorUsage: true);
}
// If the class is never instantiated, we know nothing of it can possibly
@@ -382,13 +377,11 @@ class CodegenEnqueuer implements Enqueuer {
/// that none of its methods are reflectable, unless reflectable by
/// inheritance.
void enqueueReflectiveSpecialClasses() {
- Iterable<ClassElement> classes =
- backend.classesRequiredForReflection;
+ Iterable<ClassElement> classes = backend.classesRequiredForReflection;
for (ClassElement cls in classes) {
if (backend.referencedFromMirrorSystem(cls)) {
logEnqueueReflectiveAction(cls);
- backend.registerInstantiatedType(
- cls.rawType, this, mirrorDependencies,
+ backend.registerInstantiatedType(cls.rawType, this, mirrorDependencies,
mirrorUsage: true);
}
}
« no previous file with comments | « pkg/compiler/lib/src/js_backend/backend.dart ('k') | pkg/compiler/lib/src/kernel/accessors.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698