Add field nested_sites to AllocationSite.

Add field nested_sites to AllocationSite. This field is used to maintain
allocation site information for nested array and object literals.
It's not used productively in this CL, merely maintained in a minimal
way. (that comes next :)).

BUG=
R=hpayer@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=16912

[mvstanton, 2013-09-20 12:57:02]

Hi Hannes, here is the field needed for nested literals, PTAL, thanks!
--Michael

[Hannes Payer (out of office), 2013-09-24 08:04:40]

LGTM, few nits.

https://codereview.chromium.org/23463047/diff/3001/src/heap-snapshot-generato...
File src/heap-snapshot-generator.cc (right):

https://codereview.chromium.org/23463047/diff/3001/src/heap-snapshot-generato...
src/heap-snapshot-generator.cc:1304: SetInternalReference(site, entry,
"nested_sites", site->nested_sites(),
same same

https://codereview.chromium.org/23463047/diff/3001/src/objects-printer.cc
File src/objects-printer.cc (right):

https://codereview.chromium.org/23463047/diff/3001/src/objects-printer.cc#new...
src/objects-printer.cc:1106: // Just print the number of nested sites
After discussing offline, let's print the nested site pointer and not the number
of nested sites.

https://codereview.chromium.org/23463047/diff/3001/src/objects.h
File src/objects.h (right):

https://codereview.chromium.org/23463047/diff/3001/src/objects.h#newcode7830
src/objects.h:7830: DECL_ACCESSORS(nested_sites, Object)
I think we should call it next or nested_sit and say in a comment that this list
represents the nested literals in a specific order.

https://codereview.chromium.org/23463047/diff/3001/src/objects.h#newcode7865
src/objects.h:7865: static const int kNestedSitesOffset = kTransitionInfoOffset
+ kPointerSize;
kNextOffset or kNestedSiteOffset

[mvstanton, 2013-09-24 08:39:15]

Thanks Hannes!
--Michael

https://codereview.chromium.org/23463047/diff/3001/src/objects-printer.cc
File src/objects-printer.cc (right):

https://codereview.chromium.org/23463047/diff/3001/src/objects-printer.cc#new...
src/objects-printer.cc:1106: // Just print the number of nested sites
On 2013/09/24 08:04:40, Hannes Payer wrote:
> After discussing offline, let's print the nested site pointer and not the
number
> of nested sites.

Done.

https://codereview.chromium.org/23463047/diff/3001/src/objects.h
File src/objects.h (right):

https://codereview.chromium.org/23463047/diff/3001/src/objects.h#newcode7830
src/objects.h:7830: DECL_ACCESSORS(nested_sites, Object)
On 2013/09/24 08:04:40, Hannes Payer wrote:
> I think we should call it next or nested_sit and say in a comment that this
list
> represents the nested literals in a specific order.

Done.

https://codereview.chromium.org/23463047/diff/3001/src/objects.h#newcode7865
src/objects.h:7865: static const int kNestedSitesOffset = kTransitionInfoOffset
+ kPointerSize;
On 2013/09/24 08:04:40, Hannes Payer wrote:
> kNextOffset or kNestedSiteOffset

Done.

[mvstanton, 2013-09-24 10:30:51]

Committed patchset #4 manually as r16912 (presubmit successful).