Bugfix in hydrogen array literal code generation.

Bugfix in hydrogen array literal code generation.

If an array literal contains some non-constant elements, is of type SMI, and
then the boilerplate transitions to double or fast sometime after we've
crankshafted the code, then we could incorrectly store smis in double arrays.

BUG=
R=yangguo@chromium.org

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

[mvstanton, 2013-06-19 12:32:02]

Hi Yang, here is the change we discussed and a new test file. There is actually
a test failing in mjsunit/array-literal-transitions.js that I need to solve,
though it's likely just adjusting expectations.

[Yang, 2013-06-19 12:35:05]

On 2013/06/19 12:32:02, mvstanton wrote:
> Hi Yang, here is the change we discussed and a new test file. There is
actually
> a test failing in mjsunit/array-literal-transitions.js that I need to solve,
> though it's likely just adjusting expectations.

LGTM once mjsunit/array-literal-transitions.js is fixed.

[Yang, 2013-06-19 13:40:16]

On 2013/06/19 12:35:05, Yang wrote:
> On 2013/06/19 12:32:02, mvstanton wrote:
> > Hi Yang, here is the change we discussed and a new test file. There is
> actually
> > a test failing in mjsunit/array-literal-transitions.js that I need to solve,
> > though it's likely just adjusting expectations.
> 
> LGTM once mjsunit/array-literal-transitions.js is fixed.

LGTM with a nit.

[Yang, 2013-06-19 13:40:30]

... and here's the nit.

https://codereview.chromium.org/17334004/diff/9001/src/hydrogen.cc
File src/hydrogen.cc (right):

https://codereview.chromium.org/17334004/diff/9001/src/hydrogen.cc#newcode6068
src/hydrogen.cc:6068: // De-opt if elements kind changed from
boilerplate_elements_kind
add a period at the end of the comment.

[mvstanton, 2013-06-19 13:46:51]

right on, thx, checking in..

https://codereview.chromium.org/17334004/diff/9001/src/hydrogen.cc
File src/hydrogen.cc (right):

https://codereview.chromium.org/17334004/diff/9001/src/hydrogen.cc#newcode6068
src/hydrogen.cc:6068: // De-opt if elements kind changed from
boilerplate_elements_kind
On 2013/06/19 13:40:30, Yang wrote:
> add a period at the end of the comment.

Done.

[mvstanton, 2013-06-19 13:48:57]

Committed patchset #5 manually as r15207 (presubmit successful).