Add primitive to create closures and use it for closure conversion

pkg/kernel/lib/ast.dart

Index: pkg/kernel/lib/ast.dart
diff --git a/pkg/kernel/lib/ast.dart b/pkg/kernel/lib/ast.dart
index a27c198a9326d3d6d21fcbdd261217bed35c2d67..508130ebff056f97276928bbd12e5ebf8d01e44d 100644
--- a/pkg/kernel/lib/ast.dart
+++ b/pkg/kernel/lib/ast.dart
@@ -2855,6 +2855,38 @@ class VectorCopy extends Expression {
   }
 }
 
+/// Expression of the form `MakeClosure(f, c, t)` where `f` is a name of a
+/// closed top-level function, `c` is a Vector representing closure context, and
+/// `t` is the type of the resulting closure.
+class ClosureCreation extends Expression {
+  Reference topLevelFunctionReference;

[asgerf, 2017/03/31 12:08:43]

Great!

To be consistent with the other AST nodes:
- Add a getter/setter pair for direct access to the Member
- Let the constructor take the Member as argument
- Add a .byReference constructor.

See the StaticGet class for an example.

[Dmitry Stefantsov, 2017/03/31 12:40:18]

Good idea! Obviously, I didn't payed attention to this pattern. Fixed. Thanks!

+  Expression contextVector;
+  FunctionType functionType;
+
+  ClosureCreation(
+      this.topLevelFunctionReference, this.contextVector, this.functionType) {
+    contextVector?.parent = this;
+  }
+
+  accept(ExpressionVisitor v) => v.visitClosureCreation(this);
+  accept1(ExpressionVisitor1 v, arg) => v.visitClosureCreation(this, arg);
+
+  visitChildren(Visitor v) {
+    contextVector?.accept(v);
+  }
+
+  transformChildren(Transformer v) {
+    if (contextVector != null) {
+      contextVector = contextVector.accept(v);
+      contextVector?.parent = this;
+    }
+  }
+
+  DartType getStaticType(TypeEnvironment types) {
+    return functionType;
+  }
+}
+
 // ------------------------------------------------------------------------
 //                              STATEMENTS
 // ------------------------------------------------------------------------