Reviewers: alph, loislo, Michael Starzinger,
Description:
Do not put allocated block into HeapObjectsMap
Heap allocation reported to AllocationTracker may be later divided into
several
objects so it is incorrect to put the block as a new HeapObject into the
map. We
will match allocated block with actual HeapObjects later when iterating Heap
(will be addressed in another patch).
Removed hooks for tracking creation of AllocationMemento. AllocationMemento
is
not a HeapObject and should be considered as implementation overhead.
Renamed NewObjectEvent to AllocationEvent which is more precise in case of
folded allocations and when a part of the new block becomes
AllocationMemento.
BUG=None
LOG=N
Please review this at https://codereview.chromium.org/95283003/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+46, -58 lines):
src/allocation-tracker.h
src/allocation-tracker.cc
src/builtins.cc
src/heap-inl.h
src/heap-profiler.h
src/heap-profiler.cc
src/heap-snapshot-generator.h
src/heap-snapshot-generator.cc
src/heap.cc
src/serialize.h
test/cctest/test-heap-profiler.cc
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
