AllocationSite objects weakly linked for traversal

src/objects-visiting-inl.h

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 216 matching lines @@
                                           kVisitDataObject,
                                           kVisitDataObjectGeneric>();
 
   table_.template RegisterSpecializations<JSObjectVisitor,
                                           kVisitJSObject,
                                           kVisitJSObjectGeneric>();
 
   table_.template RegisterSpecializations<StructObjectVisitor,
                                           kVisitStruct,
                                           kVisitStructGeneric>();
+
+  table_.Register(kVisitAllocationSite, &VisitAllocationSite);

[Michael Starzinger, 2013/07/16 18:14:46]

nit: Move the registration of this visitor up to after kVisitNativeContext for
consistency.

[mvstanton, 2013/07/16 19:34:57]

Done.

 }
 
 
 template<typename StaticVisitor>
 void StaticMarkingVisitor<StaticVisitor>::VisitCodeEntry(
     Heap* heap, Address entry_address) {
   Code* code = Code::cast(Code::GetObjectFromEntryAddress(entry_address));
   heap->mark_compact_collector()->RecordCodeEntrySlot(entry_address, code);
   StaticVisitor::MarkObject(heap, code);
 }
@@ skipping 79 matching lines @@
        idx < Context::NATIVE_CONTEXT_SLOTS;
        ++idx) {
     Object** slot =
         HeapObject::RawField(object, FixedArray::OffsetOfElementAt(idx));
     collector->RecordSlot(slot, slot, *slot);
   }
 }
 
 
 template<typename StaticVisitor>
+void StaticMarkingVisitor<StaticVisitor>::VisitAllocationSite(

[Michael Starzinger, 2013/07/16 18:14:46]

Having this separate visitor function is obsolete, we can register the
FixedBodyVisitor directly while initializing.

[mvstanton, 2013/07/16 19:34:57]

Done.

+    Map* map, HeapObject* object) {
+  FixedBodyVisitor<StaticVisitor,
+                   AllocationSite::BodyDescriptor,
+                   void>::Visit(map, object);
+}
+
+
+template<typename StaticVisitor>
 void StaticMarkingVisitor<StaticVisitor>::VisitMap(
     Map* map, HeapObject* object) {
   Heap* heap = map->GetHeap();
   Map* map_object = Map::cast(object);
 
   // Clears the cache of ICs related to this map.
   if (FLAG_cleanup_code_caches_at_gc) {
     map_object->ClearCodeCache(heap);
   }
 
@@ skipping 527 matching lines @@
   RelocIterator it(this, mode_mask);
   for (; !it.done(); it.next()) {
     it.rinfo()->template Visit<StaticVisitor>(heap);
   }
 }
 
 
 } }  // namespace v8::internal
 
 #endif  // V8_OBJECTS_VISITING_INL_H_