Copier should copy InvokeMethod calling convention

pkg/compiler/lib/src/cps_ir/cps_ir_nodes.dart

Index: pkg/compiler/lib/src/cps_ir/cps_ir_nodes.dart
diff --git a/pkg/compiler/lib/src/cps_ir/cps_ir_nodes.dart b/pkg/compiler/lib/src/cps_ir/cps_ir_nodes.dart
index e34a4a482a30b809c332d1227216d1c7c7ebad25..f60337c71ad9972f7e151213f72820bb58f83e7f 100644
--- a/pkg/compiler/lib/src/cps_ir/cps_ir_nodes.dart
+++ b/pkg/compiler/lib/src/cps_ir/cps_ir_nodes.dart
@@ -2483,7 +2483,8 @@ class DefinitionCopyingVisitor extends Visitor<Definition> {
   Definition visitInvokeMethod(InvokeMethod node) {
     return new InvokeMethod(getCopy(node.receiver), node.selector, node.mask,
         getList(node.arguments),
-        node.sourceInformation);
+        node.sourceInformation)
+        ..callingConvention = node.callingConvention;

[asgerf, 2016/01/06 15:38:33]

While you're at it, could you add this as a named argument to the constructor?
(cps_ir_fragment also does this in one place)

I have (with partial success) tried to uphold the convention that the fields
which are part of the concrete semantics can be passed in the constructor (as
opposed to derived properties like the type).

[sra1, 2016/01/06 20:42:04]

Done.

   }
 
   Definition visitInvokeMethodDirectly(InvokeMethodDirectly node) {