Nonvirtual pointer visitor in GCs

runtime/vm/raw_object.cc

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/raw_object.h"
 
 #include "vm/become.h"
 #include "vm/class_table.h"
 #include "vm/dart.h"
 #include "vm/freelist.h"
@@ skipping 201 matching lines @@
   }
   if ((instance_size != tags_size) && (tags_size != 0)) {
     FATAL3("Size mismatch: %" Pd " from class vs %" Pd " from tags %" Px "\n",
            instance_size, tags_size, tags);
   }
 #endif  // DEBUG
   return instance_size;
 }
 
 
-intptr_t RawObject::VisitPointers(ObjectPointerVisitor* visitor) {
+intptr_t RawObject::VisitPointersPredefined(ObjectPointerVisitor* visitor,
+                                            intptr_t class_id) {
+  ASSERT(class_id < kNumPredefinedCids);
+
   intptr_t size = 0;
 
   // Only reasonable to be called on heap objects.
   ASSERT(IsHeapObject());
 
-  // Read the necessary data out of the class before visting the class itself.
-  intptr_t class_id = GetClassId();
-
-  if (class_id < kNumPredefinedCids) {
-    switch (class_id) {
+  switch (class_id) {
 #define RAW_VISITPOINTERS(clazz)                                               \
   case k##clazz##Cid: {                                                        \
     Raw##clazz* raw_obj = reinterpret_cast<Raw##clazz*>(this);                 \
     size = Raw##clazz::Visit##clazz##Pointers(raw_obj, visitor);               \
     break;                                                                     \
   }
-      CLASS_LIST_NO_OBJECT(RAW_VISITPOINTERS)
+    CLASS_LIST_NO_OBJECT(RAW_VISITPOINTERS)
 #undef RAW_VISITPOINTERS
 #define RAW_VISITPOINTERS(clazz) case kTypedData##clazz##Cid:
-      CLASS_LIST_TYPED_DATA(RAW_VISITPOINTERS) {
-        RawTypedData* raw_obj = reinterpret_cast<RawTypedData*>(this);
-        size = RawTypedData::VisitTypedDataPointers(raw_obj, visitor);
-        break;
-      }
+    CLASS_LIST_TYPED_DATA(RAW_VISITPOINTERS) {
+      RawTypedData* raw_obj = reinterpret_cast<RawTypedData*>(this);
+      size = RawTypedData::VisitTypedDataPointers(raw_obj, visitor);
+      break;
+    }
 #undef RAW_VISITPOINTERS
 #define RAW_VISITPOINTERS(clazz) case kExternalTypedData##clazz##Cid:
-      CLASS_LIST_TYPED_DATA(RAW_VISITPOINTERS) {
-        RawExternalTypedData* raw_obj =
-            reinterpret_cast<RawExternalTypedData*>(this);
-        size = RawExternalTypedData::VisitExternalTypedDataPointers(raw_obj,
-                                                                    visitor);
-        break;
-      }
+    CLASS_LIST_TYPED_DATA(RAW_VISITPOINTERS) {
+      RawExternalTypedData* raw_obj =
+          reinterpret_cast<RawExternalTypedData*>(this);
+      size = RawExternalTypedData::VisitExternalTypedDataPointers(raw_obj,
+                                                                  visitor);
+      break;
+    }
 #undef RAW_VISITPOINTERS
 #define RAW_VISITPOINTERS(clazz) case kTypedData##clazz##ViewCid:
-      CLASS_LIST_TYPED_DATA(RAW_VISITPOINTERS)
-      case kByteDataViewCid:
-      case kByteBufferCid: {
-        RawInstance* raw_obj = reinterpret_cast<RawInstance*>(this);
-        size = RawInstance::VisitInstancePointers(raw_obj, visitor);
-        break;
-      }
+    CLASS_LIST_TYPED_DATA(RAW_VISITPOINTERS)
+    case kByteDataViewCid:
+    case kByteBufferCid: {
+      RawInstance* raw_obj = reinterpret_cast<RawInstance*>(this);
+      size = RawInstance::VisitInstancePointers(raw_obj, visitor);
+      break;
+    }
 #undef RAW_VISITPOINTERS
-      case kFreeListElement: {
-        uword addr = RawObject::ToAddr(this);
-        FreeListElement* element = reinterpret_cast<FreeListElement*>(addr);
-        size = element->Size();
-        break;
-      }
-      case kForwardingCorpse: {
-        uword addr = RawObject::ToAddr(this);
-        ForwardingCorpse* forwarder = reinterpret_cast<ForwardingCorpse*>(addr);
-        size = forwarder->Size();
-        break;
-      }
-      case kNullCid:
-        size = Size();
-        break;
-      default:
-        OS::Print("Class Id: %" Pd "\n", class_id);
-        UNREACHABLE();
-        break;
+    case kFreeListElement: {
+      uword addr = RawObject::ToAddr(this);
+      FreeListElement* element = reinterpret_cast<FreeListElement*>(addr);
+      size = element->Size();
+      break;
     }
-  } else {
-    RawInstance* raw_obj = reinterpret_cast<RawInstance*>(this);
-    size = RawInstance::VisitInstancePointers(raw_obj, visitor);
+    case kForwardingCorpse: {
+      uword addr = RawObject::ToAddr(this);
+      ForwardingCorpse* forwarder = reinterpret_cast<ForwardingCorpse*>(addr);
+      size = forwarder->Size();
+      break;
+    }
+    case kNullCid:
+      size = Size();
+      break;
+    default:
+      OS::Print("Class Id: %" Pd "\n", class_id);
+      UNREACHABLE();
+      break;
   }
 
   ASSERT(size != 0);
   ASSERT(size == Size());
   return size;
 }
 
 
 bool RawObject::FindObject(FindObjectVisitor* visitor) {
   ASSERT(visitor != NULL);
@@ skipping 709 matching lines @@
 intptr_t RawUserTag::VisitUserTagPointers(RawUserTag* raw_obj,
                                           ObjectPointerVisitor* visitor) {
   // Make sure that we got here with the tagged pointer as this.
   ASSERT(raw_obj->IsHeapObject());
   visitor->VisitPointers(raw_obj->from(), raw_obj->to());
   return UserTag::InstanceSize();
 }
 
 
 }  // namespace dart