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

Unified Diff: pkg/compiler/lib/src/dump_info.dart

Issue 2314703002: Split World usage into open, inference, and closed world. (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/compiler.dart ('k') | pkg/compiler/lib/src/elements/elements.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/compiler/lib/src/dump_info.dart
diff --git a/pkg/compiler/lib/src/dump_info.dart b/pkg/compiler/lib/src/dump_info.dart
index d5314b053e02fa9769517e2deba63bef2ff3a47b..f0d91a4e4576f29f96f8b0495b42396d10ec0eaa 100644
--- a/pkg/compiler/lib/src/dump_info.dart
+++ b/pkg/compiler/lib/src/dump_info.dart
@@ -266,12 +266,13 @@ class ElementInfoCollector extends BaseElementVisitor<Info, dynamic> {
// TODO(sigmund): why all these checks?
if (element.isInstanceMember &&
!element.isAbstract &&
- compiler.world.allFunctions.contains(element)) {
+ compiler.closedWorld.allFunctions.contains(element)) {
returnType = '${element.type.returnType}';
}
String inferredReturnType =
'${compiler.globalInference.getGuaranteedReturnTypeOfElement(element)}';
- String sideEffects = '${compiler.world.getSideEffectsOfElement(element)}';
+ String sideEffects =
+ '${compiler.closedWorld.getSideEffectsOfElement(element)}';
int inlinedCount = compiler.dumpInfoTask.inlineCount[element];
if (inlinedCount == null) inlinedCount = 0;
@@ -438,7 +439,7 @@ class DumpInfoTask extends CompilerTask implements InfoReporter {
element,
impact,
new WorldImpactVisitorImpl(visitDynamicUse: (dynamicUse) {
- selections.addAll(compiler.world.allFunctions
+ selections.addAll(compiler.closedWorld.allFunctions
.filter(dynamicUse.selector, dynamicUse.mask)
.map((e) => new Selection(e, dynamicUse.mask)));
}, visitStaticUse: (staticUse) {
« no previous file with comments | « pkg/compiler/lib/src/compiler.dart ('k') | pkg/compiler/lib/src/elements/elements.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698