Inline allocation of implicit closures.

runtime/vm/object.h

Index: runtime/vm/object.h
===================================================================
--- runtime/vm/object.h	(revision 348)
+++ runtime/vm/object.h	(working copy)
@@ -1139,11 +1139,15 @@
     return TestType(kIsAssignableTo, dst);
   }
 
-  // Returns true if this function represents a closure function.
+  // Returns true if this function represents a (possibly implicit) closure
+  // function.
   bool IsClosureFunction() const {
     return kind() == RawFunction::kClosureFunction;
   }
 
+  // Returns true if this function represents an implicit closure function.
+  bool IsImplicitClosureFunction() const;

[siva, 2011/10/11 21:00:46]

I am wondering if it would be more readable if we
had
bool IsRegularClosureFunction() const;
bool IsStaticImplicitClosureFunction() const;
bool IsDynamicImplicitClosureFunction() const;

[regis, 2011/10/11 23:40:19]

I'd like to keep IsClosureFunction() to return true for all forms, otherwise you
need to OR all forms.

I do not like "Dynamic" (because of the coming renaming of var to Dynamic), but
prefer "Instance".

I added helpers as you suggested, bu with slightly different names.

+
   // Returns true if this function represents a local function.
   bool IsLocalFunction() const {
     return parent_function() != Function::null();