Use FixedTypedArrayBase's body descriptor for static visiting

src/heap/scavenger-inl.h

 // Copyright 2015 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.
 
 #ifndef V8_HEAP_SCAVENGER_INL_H_
 #define V8_HEAP_SCAVENGER_INL_H_
 
 #include "src/heap/scavenger.h"
 
 namespace v8 {
@@ skipping 20 matching lines @@
   Heap::UpdateAllocationSiteFeedback(object, Heap::IGNORE_SCRATCHPAD_SLOT);
 
   // AllocationMementos are unrooted and shouldn't survive a scavenge
   DCHECK(object->map() != object->GetHeap()->allocation_memento_map());
   // Call the slow part of scavenge object.
   return ScavengeObjectSlow(p, object);
 }
 
 
 // static
 void StaticScavengeVisitor::VisitPointer(Heap* heap, Object** p) {

[Igor Sheludko, 2015/12/01 15:22:45]

What about removing this old one? Given that all these functions must be inlined
anyway the extra |obj| parameter should be optimized away by the compiler.

[jochen (gone - plz use gerrit), 2015/12/01 15:29:28]

not sure what you mean?

   Object* object = *p;
   if (!heap->InNewSpace(object)) return;
   Scavenger::ScavengeObject(reinterpret_cast<HeapObject**>(p),
                             reinterpret_cast<HeapObject*>(object));
 }
 
+
+// static
+void StaticScavengeVisitor::VisitPointer(Heap* heap, HeapObject* obj,
+                                         Object** p) {
+  Object* object = *p;
+  if (!heap->InNewSpace(object)) return;
+  Scavenger::ScavengeObject(reinterpret_cast<HeapObject**>(p),
+                            reinterpret_cast<HeapObject*>(object));
+}
+
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_HEAP_SCAVENGER_INL_H_