Create Namer and Emitter on codegen start.

pkg/compiler/lib/src/js_emitter/code_emitter_task.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 part of dart2js.js_emitter;
 
 const USE_LAZY_EMITTER = const bool.fromEnvironment("dart2js.use.lazy.emitter");
 
 /**
  * Generates the code for all used classes in the program. Static fields (even
  * in classes) are ignored, since they can be treated as non-class elements.
  *
  * The code for the containing (used) methods must exist in the `universe`.
  */
 class CodeEmitterTask extends CompilerTask {
-  // TODO(floitsch): the code-emitter task should not need a namer.
-  final Namer namer;
-  final TypeTestRegistry typeTestRegistry;
+  TypeTestRegistry typeTestRegistry;
   NativeEmitter nativeEmitter;
   MetadataCollector metadataCollector;
-  Emitter emitter;
+  EmitterFactory _emitterFactory;
+  Emitter _emitter;
   final Compiler compiler;
 
   /// Records if a type variable is read dynamically for type tests.
   final Set<TypeVariableElement> readTypeVariables =
       new Set<TypeVariableElement>();
 
   JavaScriptBackend get backend => compiler.backend;
 
   @deprecated
   // This field should be removed. It's currently only needed for dump-info and
   // tests.
   // The field is set after the program has been emitted.
   /// Contains a list of all classes that are emitted.
   Set<ClassElement> neededClasses;
 
-  CodeEmitterTask(Compiler compiler, Namer namer, bool generateSourceMap,
-      bool useStartupEmitter)
+  CodeEmitterTask(
+      Compiler compiler, bool generateSourceMap, bool useStartupEmitter)
       : compiler = compiler,
-        this.namer = namer,
-        this.typeTestRegistry = new TypeTestRegistry(compiler),
         super(compiler.measurer) {
     nativeEmitter = new NativeEmitter(this);
     if (USE_LAZY_EMITTER) {
-      emitter = new lazy_js_emitter.Emitter(compiler, namer, nativeEmitter);
+      _emitterFactory = new lazy_js_emitter.EmitterFactory();
     } else if (useStartupEmitter) {
-      emitter = new startup_js_emitter.Emitter(
-          compiler, namer, nativeEmitter, generateSourceMap);
+      _emitterFactory = new startup_js_emitter.EmitterFactory(
+          generateSourceMap: generateSourceMap);
     } else {
-      emitter =
-          new full_js_emitter.Emitter(compiler, namer, generateSourceMap, this);
+      _emitterFactory = new full_js_emitter.EmitterFactory(
+          generateSourceMap: generateSourceMap);
     }
-    metadataCollector = new MetadataCollector(compiler, emitter);
+  }
+
+  Emitter get emitter {
+    assert(invariant(NO_LOCATION_SPANNABLE, _emitter != null,
+        message: "Emitter has not been created yet."));
+    return _emitter;
   }
 
   String get name => 'Code emitter';
 
   /// Returns the string that is used to find library patches that are
   /// specialized for the emitter.
-  String get patchVersion => emitter.patchVersion;
+  String get patchVersion => _emitterFactory.patchVersion;
+
+  /// Returns true, if the emitter supports reflection.
+  bool get supportsReflection => _emitterFactory.supportsReflection;
 
   /// Returns the closure expression of a static function.
   jsAst.Expression isolateStaticClosureAccess(MethodElement element) {
     return emitter.isolateStaticClosureAccess(element);
   }
 
   /// Returns the JS function that must be invoked to get the value of the
   /// lazily initialized static.
   jsAst.Expression isolateLazyInitializerAccess(FieldElement element) {
     return emitter.isolateLazyInitializerAccess(element);
@@ skipping 60 matching lines @@
   }
 
   Set<ClassElement> _finalizeRti() {
     // Compute the required type checks to know which classes need a
     // 'is$' method.
     typeTestRegistry.computeRequiredTypeChecks();
     // Compute the classes needed by RTI.
     return typeTestRegistry.computeRtiNeededClasses();
   }
 
-  int assembleProgram() {
+  /// Creates the [Emitter] for this task.
+  void createEmitter(Namer namer, ClosedWorld closedWorld) {
+    measure(() {
+      _emitter = _emitterFactory.createEmitter(this, namer, closedWorld);
+      metadataCollector = new MetadataCollector(compiler, _emitter);
+      typeTestRegistry = new TypeTestRegistry(compiler, closedWorld);
+    });
+  }
+
+  int assembleProgram(Namer namer, ClosedWorld closedWorld) {
     return measure(() {
       emitter.invalidateCaches();
 
       Set<ClassElement> rtiNeededClasses = _finalizeRti();
-      ProgramBuilder programBuilder =
-          new ProgramBuilder(compiler, namer, this, emitter, rtiNeededClasses);
+      ProgramBuilder programBuilder = new ProgramBuilder(
+          compiler, namer, this, emitter, closedWorld, rtiNeededClasses);
       int size = emitter.emitProgram(programBuilder);
       // TODO(floitsch): we shouldn't need the `neededClasses` anymore.
       neededClasses = programBuilder.collector.neededClasses;
       return size;
     });
   }
 }
 
-abstract class Emitter {
+abstract class EmitterFactory {
   /// Returns the string that is used to find library patches that are
   /// specialized for this emitter.
   String get patchVersion;
 
+  /// Returns true, if the emitter supports reflection.
+  bool get supportsReflection;
+
+  /// Create the [Emitter] for the emitter [task] that uses the given [namer].
+  Emitter createEmitter(
+      CodeEmitterTask task, Namer namer, ClosedWorld closedWorld);
+}
+
+abstract class Emitter {
   /// Uses the [programBuilder] to generate a model of the program, emits
   /// the program, and returns the size of the generated output.
   int emitProgram(ProgramBuilder programBuilder);
 
-  /// Returns true, if the emitter supports reflection.
-  bool get supportsReflection;
-
   /// Returns the JS function that must be invoked to get the value of the
   /// lazily initialized static.
   jsAst.Expression isolateLazyInitializerAccess(FieldElement element);
 
   /// Returns the closure expression of a static function.
   jsAst.Expression isolateStaticClosureAccess(FunctionElement element);
 
   /// Returns the JS code for accessing the embedded [global].
   jsAst.Expression generateEmbeddedGlobalAccess(String global);
 
@@ skipping 25 matching lines @@
   bool isConstantInlinedOrAlreadyEmitted(ConstantValue constant);
 
   /// Returns the JS code for accessing the given [constant].
   jsAst.Expression constantReference(ConstantValue constant);
 
   /// Returns the JS template for the given [builtin].
   jsAst.Template templateForBuiltin(JsBuiltin builtin);
 
   void invalidateCaches();
 }