Add support for sync* and yield

runtime/lib/function.cc

Index: runtime/lib/function.cc
===================================================================
--- runtime/lib/function.cc	(revision 43415)
+++ runtime/lib/function.cc	(working copy)
@@ -74,4 +74,28 @@
   return Object::null();
 }
 
+
+DEFINE_NATIVE_ENTRY(FunctionImpl_clone, 1) {
+  const Instance& receiver = Instance::CheckedHandle(
+      isolate, arguments->NativeArgAt(0));
+  ASSERT(receiver.IsClosure());
+  if (receiver.IsClosure()) {
+    const Function& func =
+        Function::Handle(isolate, Closure::function(receiver));
+    const Context& ctx =
+        Context::Handle(isolate, Closure::context(receiver));
+    Context& cloned_ctx =
+        Context::Handle(isolate, Context::New(ctx.num_variables()));
+    cloned_ctx.set_parent(Context::Handle(isolate, ctx.parent()));
+    Object& inst = Object::Handle(isolate);
+    for (int i = 0; i < ctx.num_variables(); i++) {
+      inst = ctx.At(i);
+      cloned_ctx.SetAt(i, inst);
+    }
+    return Closure::New(func, cloned_ctx);
+  }
+  return Object::null();

[srdjan, 2015/02/04 19:29:12]

Should that be UNREACHABLE()?

[hausner, 2015/02/04 22:09:24]

I modeled this after FunctionImpl_equals above. There is an assertion that the
parameter is a closure, so we do have a check in debug mode. I don't have a
strong preference.

+}
+
+
 }  // namespace dart