Remove superfluous HInnerAllocatedObject in BuildEmitDeepCopy

Remove superfluous HInnerAllocatedObject in BuildEmitDeepCopy

R=hpayer@chromium.org

Committed: http://code.google.com/p/v8/source/detail?r=15367

[Toon Verwaest, 2013-06-27 04:32:22]

Dbc

https://codereview.chromium.org/17881004/diff/1/src/hydrogen.cc
File src/hydrogen.cc (right):

https://codereview.chromium.org/17881004/diff/1/src/hydrogen.cc#newcode10033
src/hydrogen.cc:10033: }
Isn't there a utility function to test for this condition? If not, add one to
jsobject or map. Specifically to test for fields you can also look at the unused
property fields, inobject properties etc to check this condition in O(1).

[danno, 2013-06-27 13:55:51]

Please take another look

https://codereview.chromium.org/17881004/diff/1/src/hydrogen.cc
File src/hydrogen.cc (right):

https://codereview.chromium.org/17881004/diff/1/src/hydrogen.cc#newcode10033
src/hydrogen.cc:10033: }
On 2013/06/27 04:32:22, Toon Verwaest wrote:
> Isn't there a utility function to test for this condition? If not, add one to
> jsobject or map. Specifically to test for fields you can also look at the
unused
> property fields, inobject properties etc to check this condition in O(1).

OK, I now use map()->NumberOfFields(). I'll leave it to you to optimize
NumberOfFields to use a O(1) algorithm :-)

[Hannes Payer (out of office), 2013-06-27 16:04:15]

lgtm

[danno, 2013-06-27 16:08:01]

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