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

Unified Diff: tests/compiler/dart2js/kernel/impact_test.dart

Issue 2486703002: Change IR encoding of redirecting factory invocations. (Closed)
Patch Set: Fix impact_test. Created 4 years, 1 month 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/kernel/kernel_visitor.dart ('k') | tests/compiler/dart2js/kernel/visitor_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/compiler/dart2js/kernel/impact_test.dart
diff --git a/tests/compiler/dart2js/kernel/impact_test.dart b/tests/compiler/dart2js/kernel/impact_test.dart
index 363f44d23666ece8b2297f8727b8365cd17a3579..fd2467f8ffeca1c2b38a7f5e24549f7e98444d74 100644
--- a/tests/compiler/dart2js/kernel/impact_test.dart
+++ b/tests/compiler/dart2js/kernel/impact_test.dart
@@ -688,13 +688,18 @@ ResolutionImpact laxImpact(
new ResolutionWorldImpactBuilder('Lax impact of ${element}');
for (StaticUse staticUse in impact.staticUses) {
switch (staticUse.kind) {
+ case StaticUseKind.CONSTRUCTOR_INVOKE:
case StaticUseKind.CONST_CONSTRUCTOR_INVOKE:
ConstructorElement constructor = staticUse.element;
ConstructorElement effectiveTarget = constructor.effectiveTarget;
DartType effectiveTargetType =
constructor.computeEffectiveTargetType(staticUse.type);
- builder.registerStaticUse(new StaticUse.constConstructorInvoke(
- effectiveTarget.declaration, null, effectiveTargetType));
+ builder.registerStaticUse(
+ staticUse.kind == StaticUseKind.CONST_CONSTRUCTOR_INVOKE
+ ? new StaticUse.constConstructorInvoke(
+ effectiveTarget.declaration, null, effectiveTargetType)
+ : new StaticUse.typedConstructorInvoke(
+ effectiveTarget.declaration, null, effectiveTargetType));
break;
default:
builder.registerStaticUse(staticUse);
« no previous file with comments | « pkg/compiler/lib/src/kernel/kernel_visitor.dart ('k') | tests/compiler/dart2js/kernel/visitor_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698