Allocation sampling for paged/lo spaces

src/heap/spaces-inl.h

 // Copyright 2011 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_SPACES_INL_H_
 #define V8_HEAP_SPACES_INL_H_
 
 #include "src/heap/incremental-marking.h"
 #include "src/heap/spaces.h"
 #include "src/isolate.h"
@@ skipping 407 matching lines @@
     return object;
   }
 
   return AllocationResult::Retry(identity());
 }
 
 
 AllocationResult PagedSpace::AllocateRaw(int size_in_bytes,
                                          AllocationAlignment alignment) {
 #ifdef V8_HOST_ARCH_32_BIT
-  return alignment == kDoubleAligned
-             ? AllocateRawAligned(size_in_bytes, kDoubleAligned)
-             : AllocateRawUnaligned(size_in_bytes);
+  AllocationResult result =
+      alignment == kDoubleAligned
+          ? AllocateRawAligned(size_in_bytes, kDoubleAligned)
+          : AllocateRawUnaligned(size_in_bytes);
 #else
-  return AllocateRawUnaligned(size_in_bytes);
+  AllocationResult result = AllocateRawUnaligned(size_in_bytes);
 #endif
+  if (!result.IsRetry()) {
+    Object* obj = result.ToObjectChecked();

[ofrobots, 2016/02/11 05:51:46]

This code would be much simpler if you used AllocationResult::To<T> template
function instead. See example here:
https://code.google.com/p/chromium/codesearch#chromium/src/v8/src/heap/heap-i...

[mattloring, 2016/02/11 14:54:14]

Done.

+    if (obj->IsHeapObject()) {
+      HeapObject* heap_obj = static_cast<HeapObject*>(obj);
+      AllocationStep(heap_obj->address(), size_in_bytes);
+    }
+  }
+  return result;
 }
 
 
 // -----------------------------------------------------------------------------
 // NewSpace
 
 
 AllocationResult NewSpace::AllocateRawAligned(int size_in_bytes,
                                               AllocationAlignment alignment) {
   Address top = allocation_info_.top();
@@ skipping 100 matching lines @@
     other->allocation_info_.Reset(nullptr, nullptr);
     return true;
   }
   return false;
 }
 
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_HEAP_SPACES_INL_H_