Handlify AllocationSite AddDependentCompilationInfo.

Handlify AllocationSite AddDependentCompilationInfo.

BUG=
R=mvstanton@chromium.org

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

[mvstanton, 2014-01-13 14:37:56]

lgtm.

https://codereview.chromium.org/136703002/diff/1/src/objects.cc
File src/objects.cc (right):

https://codereview.chromium.org/136703002/diff/1/src/objects.cc#newcode12881
src/objects.cc:12881: // static
nit: I don't think we usually add // static (although it is informative). Up to
you.

[Benedikt Meurer, 2014-01-13 14:38:46]

https://codereview.chromium.org/136703002/diff/1/src/objects.cc
File src/objects.cc (right):

https://codereview.chromium.org/136703002/diff/1/src/objects.cc#newcode12898
src/objects.cc:12898: if (*codes != dependent_code())
set_dependent_code(*codes);
Please remove this method.

[Hannes Payer (out of office), 2014-01-13 14:53:20]

https://codereview.chromium.org/136703002/diff/1/src/objects.cc
File src/objects.cc (right):

https://codereview.chromium.org/136703002/diff/1/src/objects.cc#newcode12881
src/objects.cc:12881: // static
On 2014/01/13 14:37:56, mvstanton wrote:
> nit: I don't think we usually add // static (although it is informative). Up
to
> you.

That is a Chrome thing. We do it inconsistently, I will leave it in.

https://codereview.chromium.org/136703002/diff/1/src/objects.cc#newcode12898
src/objects.cc:12898: if (*codes != dependent_code())
set_dependent_code(*codes);
On 2014/01/13 14:38:47, Benedikt Meurer wrote:
> Please remove this method.

Done.

[Hannes Payer (out of office), 2014-01-13 14:54:39]

Committed patchset #2 manually as r18570 (presubmit successful).