Consolidate a few fields in RawFunction using bitfields.

runtime/vm/object.h

Index: runtime/vm/object.h
===================================================================
--- runtime/vm/object.h	(revision 9727)
+++ runtime/vm/object.h	(working copy)
@@ -1355,21 +1355,18 @@
   // If none exists yet, create one and remember it.
   RawFunction* ImplicitClosureFunction() const;
 
-  RawFunction::Kind kind() const { return raw_ptr()->kind_; }
+  RawFunction::Kind kind() const { return raw()->GetKind(); }
 
-  bool is_static() const { return raw_ptr()->is_static_; }
-  bool is_const() const { return raw_ptr()->is_const_; }
+  bool is_static() const { return raw()->IsStatic(); }
+  bool is_const() const { return raw()->IsConst(); }
   bool IsConstructor() const {
     return (kind() == RawFunction::kConstructor) && !is_static();
   }
   bool IsFactory() const {
     return (kind() == RawFunction::kConstructor) && is_static();
   }
-  bool IsAbstract() const {
-    return kind() == RawFunction::kAbstract;
-  }
   bool IsDynamicFunction() const {
-    if (is_static()) {
+    if (is_static() || is_abstract()) {

[turnidge, 2012/07/18 04:57:15]

What do you think of this?  I'm preserving the current behavior, but not sure if
it is useful.  I could drop the is_abstract() test here...

       return false;
     }
     switch (kind()) {
@@ -1382,7 +1379,6 @@
       case RawFunction::kClosureFunction:
       case RawFunction::kConstructor:
       case RawFunction::kConstImplicitGetter:
-      case RawFunction::kAbstract:
         return false;
       default:
         UNREACHABLE();
@@ -1451,14 +1447,15 @@
     raw_ptr()->deoptimization_counter_ = value;
   }
 
-  bool is_optimizable() const {
-    return raw_ptr()->is_optimizable_;
-  }
+  bool is_optimizable() const { return raw()->IsOptimizable(); }
   void set_is_optimizable(bool value) const;
 
-  bool is_native() const { return raw_ptr()->is_native_; }
+  bool is_native() const { return raw()->IsNative(); }
   void set_is_native(bool value) const;
 
+  bool is_abstract() const { return raw()->IsAbstract(); }
+  void set_is_abstract(bool value) const;
+
   bool HasOptimizedCode() const;
 
   intptr_t NumberOfParameters() const;
@@ -1557,6 +1554,7 @@
                           RawFunction::Kind kind,
                           bool is_static,
                           bool is_const,
+                          bool is_abstract,
                           intptr_t token_pos);
 
   // Allocates a new Function object representing a closure function, as well as