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

Unified Diff: dart/pkg/compiler/lib/src/js_emitter/code_emitter_task.dart

Issue 768993003: Track emitted classes in incremental compiler. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge
Patch Set: Merged with r42279. Created 6 years 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: dart/pkg/compiler/lib/src/js_emitter/code_emitter_task.dart
diff --git a/dart/pkg/compiler/lib/src/js_emitter/code_emitter_task.dart b/dart/pkg/compiler/lib/src/js_emitter/code_emitter_task.dart
index 0aaf241c14fe12bc8a725cc1381da78ac14736ad..6dcc012b766cf245b6efde8babe220ab256d4df1 100644
--- a/dart/pkg/compiler/lib/src/js_emitter/code_emitter_task.dart
+++ b/dart/pkg/compiler/lib/src/js_emitter/code_emitter_task.dart
@@ -347,21 +347,24 @@ class CodeEmitterTask extends CompilerTask {
backend.generatedCode.keys.forEach(addSurroundingLibraryToSet);
neededClasses.forEach(addSurroundingLibraryToSet);
-}
+ }
+
+ void computeAllNeededEntities() {
+ // Compute the required type checks to know which classes need a
+ // 'is$' method.
+ typeTestRegistry.computeRequiredTypeChecks();
ahe 2014/12/11 09:40:15 Conflict resolution: typeTestRegistry was renamed.
+
+ computeNeededDeclarations();
+ computeNeededConstants();
+ computeNeededStatics();
+ computeNeededLibraries();
+ }
void assembleProgram() {
measure(() {
emitter.invalidateCaches();
- // Compute the required type checks to know which classes need a
- // 'is$' method.
- typeTestRegistry.computeRequiredTypeChecks();
-
- computeNeededDeclarations();
- computeNeededConstants();
- computeNeededStatics();
- computeNeededLibraries();
-
+ computeAllNeededEntities();
Program program;
if (USE_NEW_EMITTER) {

Powered by Google App Engine
This is Rietveld 408576698