Add flag to trace object groups

src/global-handles.cc

 // Copyright 2009 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/global-handles.h"
 
 #include "src/api.h"
 #include "src/v8.h"
 #include "src/vm-state-inl.h"
 
@@ skipping 751 matching lines @@
         v->VisitPointer(node->location());
       }
     }
   }
 }
 
 
 bool GlobalHandles::IterateObjectGroups(ObjectVisitor* v,
                                         WeakSlotCallbackWithHeap can_skip) {
   ComputeObjectGroupsAndImplicitReferences();
+  if (FLAG_trace_object_groups) {
+    ObjectGroupsTracer(isolate_).CollectTraces();

[ulan, 2016/03/08 20:23:32]

Not needed anymore.

[Marcel Hlopko, 2016/03/09 09:30:30]

Done.

+  }
+
   int last = 0;
   bool any_group_was_visited = false;
   for (int i = 0; i < object_groups_.length(); i++) {
     ObjectGroup* entry = object_groups_.at(i);
     DCHECK(entry != NULL);
 
     Object*** objects = entry->objects;
     bool group_should_be_visited = false;
     for (size_t j = 0; j < entry->length; j++) {
       Object* object = *objects[j];
@@ skipping 22 matching lines @@
 
     // Once the entire group has been iterated over, set the object
     // group to NULL so it won't be processed again.
     delete entry;
     object_groups_.at(i) = NULL;
   }
   object_groups_.Rewind(last);
   return any_group_was_visited;
 }
 
+namespace {
+// Traces the information about object groups and implicit ref groups given by
+// the embedder to the V8 during each gc prologue.
+class ObjectGroupsTracer {
+ public:
+  class HasDumpObject {
+   public:
+    void DumpObject(Object* object);
+
+   private:
+    void DumpConstructor(JSObject* js_object);
+    void DumpInternalFields(JSObject* js_object);
+  };
+
+  class ObjectGroupVisitor : public HasDumpObject {
+   public:
+    explicit ObjectGroupVisitor(Isolate* isolate);
+    void Apply(ObjectGroup** group);
+
+   private:
+    Isolate* isolate_;
+  };
+
+  class ImplicitRefGroupVisitor : public HasDumpObject {
+   public:
+    explicit ImplicitRefGroupVisitor(Isolate* isolate);
+    void Apply(ImplicitRefGroup** group);
+
+   private:
+    Isolate* isolate_;
+  };
+
+  explicit ObjectGroupsTracer(Isolate* isolate);
+  void CollectTraces();
+
+ private:
+  Isolate* isolate_;
+  DISALLOW_COPY_AND_ASSIGN(ObjectGroupsTracer);
+};
+
+ObjectGroupsTracer::ObjectGroupsTracer(Isolate* isolate) : isolate_(isolate) {}
+
+void ObjectGroupsTracer::CollectTraces() {
+  GlobalHandles* global_handles = isolate_->global_handles();
+  ObjectGroupVisitor object_groups_visitor(isolate_);
+  ImplicitRefGroupVisitor implicit_ref_groups_visitor(isolate_);
+
+  PrintIsolate(isolate_, "### Tracing object groups:\n");
+
+  global_handles->object_groups()->Iterate(&object_groups_visitor);
+  global_handles->implicit_ref_groups()->Iterate(&implicit_ref_groups_visitor);
+
+  PrintIsolate(isolate_, "### Tracing object groups finished.\n");
+}
+
+void ObjectGroupsTracer::HasDumpObject::DumpObject(Object* o) {
+  JSObject* js_object = reinterpret_cast<JSObject*>(o);
+
+  PrintF("{ constructor_name: ");
+  DumpConstructor(js_object);
+  PrintF(", hidden_fields: [ ");
+  DumpInternalFields(js_object);
+  PrintF(" ] }\n");
+}
+
+void ObjectGroupsTracer::HasDumpObject::DumpConstructor(JSObject* js_object) {
+  Object* maybe_constructor = js_object->map()->GetConstructor();
+  if (maybe_constructor->IsJSFunction()) {
+    JSFunction* constructor = JSFunction::cast(maybe_constructor);
+    String* name = String::cast(constructor->shared()->name());
+    if (name->length() == 0) name = constructor->shared()->inferred_name();
+
+    PrintF("%s", name->ToCString().get());
+  } else if (maybe_constructor->IsNull()) {
+    if (js_object->IsOddball()) {
+      PrintF("<oddball>");
+    } else {
+      PrintF("<null>");
+    }
+  } else {
+    PrintF("boo will crash\n");
+    maybe_constructor->Print();
+    UNREACHABLE();
+  }
+}
+
+void ObjectGroupsTracer::HasDumpObject::DumpInternalFields(
+    JSObject* js_object) {
+  if (js_object->IsOddball()) {
+    return;
+  }
+
+  for (int i = 0; i < js_object->GetInternalFieldCount(); ++i) {
+    if (i != 0) {
+      PrintF(", ");
+    }
+    PrintF("%p", js_object->GetInternalField(i));
+  }
+}
+
+ObjectGroupsTracer::ObjectGroupVisitor::ObjectGroupVisitor(Isolate* isolate)
+    : isolate_(isolate) {}
+
+ObjectGroupsTracer::ImplicitRefGroupVisitor::ImplicitRefGroupVisitor(
+    Isolate* isolate)
+    : isolate_(isolate) {}
+
+void ObjectGroupsTracer::ObjectGroupVisitor::Apply(ObjectGroup** p) {
+  ObjectGroup* group = *p;
+  PrintIsolate(isolate_, "ObjectGroup (size: %lu)\n", group->length);
+  Object*** objects = group->objects;
+
+  for (size_t i = 0; i < group->length; ++i) {
+    PrintIsolate(isolate_, "  - Member: ");
+    DumpObject(*objects[i]);
+  }
+}
+
+void ObjectGroupsTracer::ImplicitRefGroupVisitor::Apply(ImplicitRefGroup** p) {
+  ImplicitRefGroup* group = *p;
+  PrintIsolate(isolate_, "ImplicitRefGroup (children count: %lu)\n",
+               group->length);
+  PrintIsolate(isolate_, "  - Parent: ");
+  DumpObject(*(group->parent));
+
+  Object*** children = group->children;
+  for (size_t i = 0; i < group->length; ++i) {
+    PrintIsolate(isolate_, "  - Child: ");
+    DumpObject(*children[i]);
+  }
+}
+
+}  // namespace
+
+void GlobalHandles::PrintObjectGroups() {
+  ObjectGroupsTracer(isolate_).CollectTraces();
+}
 
 void GlobalHandles::InvokeSecondPassPhantomCallbacks(
     List<PendingPhantomCallback>* callbacks, Isolate* isolate) {
   while (callbacks->length() != 0) {
     auto callback = callbacks->RemoveLast();
     DCHECK(callback.node() == nullptr);
     // Fire second pass callback
     callback.Invoke(isolate);
   }
 }
@@ skipping 553 matching lines @@
   blocks_[block][offset] = object;
   if (isolate->heap()->InNewSpace(object)) {
     new_space_indices_.Add(size_);
   }
   *index = size_++;
 }
 
 
 }  // namespace internal
 }  // namespace v8