Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file | 1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file |
| 2 // for details. All rights reserved. Use of this source code is governed by a | 2 // for details. All rights reserved. Use of this source code is governed by a |
| 3 // BSD-style license that can be found in the LICENSE file. | 3 // BSD-style license that can be found in the LICENSE file. |
| 4 | 4 |
| 5 part of dart2js; | 5 part of dart2js; |
| 6 | 6 |
| 7 class EnqueueTask extends CompilerTask { | 7 class EnqueueTask extends CompilerTask { |
| 8 final ResolutionEnqueuer resolution; | 8 final ResolutionEnqueuer resolution; |
| 9 final CodegenEnqueuer codegen; | 9 final CodegenEnqueuer codegen; |
| 10 | 10 |
| (...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 80 /** | 80 /** |
| 81 * Adds [element] to the work list if it has not already been processed. | 81 * Adds [element] to the work list if it has not already been processed. |
| 82 * | 82 * |
| 83 * Returns [:true:] if the [element] should be processed. | 83 * Returns [:true:] if the [element] should be processed. |
| 84 */ | 84 */ |
| 85 // TODO(johnniwinther): Change to 'Returns true if the element was added to | 85 // TODO(johnniwinther): Change to 'Returns true if the element was added to |
| 86 // the work list'? | 86 // the work list'? |
| 87 bool addElementToWorkList(Element element, [TreeElements elements]); | 87 bool addElementToWorkList(Element element, [TreeElements elements]); |
| 88 | 88 |
| 89 void registerInstantiatedType(InterfaceType type) { | 89 void registerInstantiatedType(InterfaceType type) { |
| 90 ClassElement cls = type.element; | |
| 91 cls.ensureResolved(compiler); | |
| 90 universe.instantiatedTypes.add(type); | 92 universe.instantiatedTypes.add(type); |
| 91 registerInstantiatedClass(type.element); | |
| 92 } | |
| 93 | |
| 94 void registerInstantiatedClass(ClassElement cls) { | |
| 95 if (universe.instantiatedClasses.contains(cls)) return; | 93 if (universe.instantiatedClasses.contains(cls)) return; |
| 96 if (!cls.isAbstract(compiler)) { | 94 if (!cls.isAbstract(compiler)) { |
| 97 universe.instantiatedClasses.add(cls); | 95 universe.instantiatedClasses.add(cls); |
| 98 } | 96 } |
| 99 onRegisterInstantiatedClass(cls); | 97 onRegisterInstantiatedClass(cls); |
| 100 compiler.backend.registerInstantiatedClass(cls, this); | 98 compiler.backend.registerInstantiatedClass(cls, this); |
| 101 } | 99 } |
| 102 | 100 |
| 101 void registerInstantiatedClass(ClassElement cls) { | |
|
ngeoffray
2013/03/05 10:14:10
Remove this method?
karlklose
2013/03/05 11:34:30
It still serves as a convenient method to register
| |
| 102 cls.ensureResolved(compiler); | |
| 103 registerInstantiatedType(cls.rawType); | |
| 104 } | |
| 105 | |
| 103 bool checkNoEnqueuedInvokedInstanceMethods() { | 106 bool checkNoEnqueuedInvokedInstanceMethods() { |
| 104 task.measure(() { | 107 task.measure(() { |
| 105 // Run through the classes and see if we need to compile methods. | 108 // Run through the classes and see if we need to compile methods. |
| 106 for (ClassElement classElement in universe.instantiatedClasses) { | 109 for (ClassElement classElement in universe.instantiatedClasses) { |
| 107 for (ClassElement currentClass = classElement; | 110 for (ClassElement currentClass = classElement; |
| 108 currentClass != null; | 111 currentClass != null; |
| 109 currentClass = currentClass.superclass) { | 112 currentClass = currentClass.superclass) { |
| 110 processInstantiatedClass(currentClass); | 113 processInstantiatedClass(currentClass); |
| 111 } | 114 } |
| 112 } | 115 } |
| (...skipping 423 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 536 while(!queue.isEmpty) { | 539 while(!queue.isEmpty) { |
| 537 // TODO(johnniwinther): Find an optimal process order for codegen. | 540 // TODO(johnniwinther): Find an optimal process order for codegen. |
| 538 f(queue.removeLast()); | 541 f(queue.removeLast()); |
| 539 } | 542 } |
| 540 } | 543 } |
| 541 | 544 |
| 542 void _logSpecificSummary(log(message)) { | 545 void _logSpecificSummary(log(message)) { |
| 543 log('Compiled ${generatedCode.length} methods.'); | 546 log('Compiled ${generatedCode.length} methods.'); |
| 544 } | 547 } |
| 545 } | 548 } |
| OLD | NEW |