Don't evaluate the argument twice for unresolved static setter calls.

sdk/lib/_internal/compiler/implementation/ssa/builder.dart

Index: sdk/lib/_internal/compiler/implementation/ssa/builder.dart
diff --git a/sdk/lib/_internal/compiler/implementation/ssa/builder.dart b/sdk/lib/_internal/compiler/implementation/ssa/builder.dart
index 70713dad882d0a1001d0ca5d9df52d3f68575730..1a595d3d102f17b83e36668f3b6bf17dcaf37575 100644
--- a/sdk/lib/_internal/compiler/implementation/ssa/builder.dart
+++ b/sdk/lib/_internal/compiler/implementation/ssa/builder.dart
@@ -2719,7 +2719,8 @@ class SsaBuilder extends ResolvedVisitor implements Visitor {
       generateThrowNoSuchMethod(
           location,
           getTargetName(element, 'set'),
-          argumentNodes: (send == null ? const Link<Node>() : send.arguments));
+          argumentValues: (send == null ? <HInstruction>[] :

[kasperl, 2013/10/04 07:30:27]

I'd compute the argument values outside the call to generateThrowNoSuchMethod
and stuff it in a local (to avoid some of the nesting). Could the empty list be
const?

[karlklose, 2013/10/04 08:46:55]

Done.

+                                          <HInstruction>[value]));
     } else {
       stack.add(value);
       // If the value does not already have a name, give it here.