Chromium Code Reviews| Index: dart/sdk/lib/_internal/compiler/implementation/ssa/builder.dart |
| diff --git a/dart/sdk/lib/_internal/compiler/implementation/ssa/builder.dart b/dart/sdk/lib/_internal/compiler/implementation/ssa/builder.dart |
| index cb5b3626d067f9b850552d8777944b4792b58cb0..15a49da01b3e65d7bd0eca3b87556c364b8944b7 100644 |
| --- a/dart/sdk/lib/_internal/compiler/implementation/ssa/builder.dart |
| +++ b/dart/sdk/lib/_internal/compiler/implementation/ssa/builder.dart |
| @@ -4040,9 +4040,11 @@ class SsaBuilder extends ResolvedVisitor implements Visitor { |
| native.handleSsaNative(this, node.expression); |
| return; |
| } |
| - assert(invariant(node, !node.isRedirectingFactoryBody)); |
| HInstruction value; |
| - if (node.expression == null) { |
| + if (node.isRedirectingFactoryBody) { |
| + // TODO(ahe): This is only for reflection, and it is not correct yet. |
| + value = graph.addConstantNull(constantSystem); |
|
ngeoffray
2013/05/27 07:53:55
How do you actually end up having a redirecting fa
ahe
2013/05/27 08:10:01
This was because of wrong translation of !(a && b)
ngeoffray
2013/05/27 08:12:37
I see, thanks for the explanation.
|
| + } else if (node.expression == null) { |
| value = graph.addConstantNull(constantSystem); |
| } else { |
| visit(node.expression); |