Parallel compilation bug: LCallNewArray dereferenced a handle unnecessarily.

Parallel compilation bug: LCallNewArray dereferenced a handle unnecessarily.

BUG=
R=mstarzinger@chromium.org

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

[mvstanton, 2013-05-08 09:29:53]

Hi Michael, here is a parallel compilation issue found when we use
[H|L]CallNewArray. The fix is to examine and store ElementsKind in
HCallNewArray.

[Michael Starzinger, 2013-05-08 10:40:30]

LGTM, with one suggestion.

https://codereview.chromium.org/14761010/diff/1/src/hydrogen-instructions.h
File src/hydrogen-instructions.h (right):

https://codereview.chromium.org/14761010/diff/1/src/hydrogen-instructions.h#n...
src/hydrogen-instructions.h:2362: ElementsKind kind() const { return kind_; }
There are so many "kind"s in V8 already, can we rename the field and the
accessor to "elements_kind" here?

[mvstanton, 2013-05-08 16:48:34]

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

[mvstanton, 2013-05-08 16:48:42]

Thanks, done!

On 2013/05/08 10:40:30, Michael Starzinger wrote:
> LGTM, with one suggestion.
> 
> https://codereview.chromium.org/14761010/diff/1/src/hydrogen-instructions.h
> File src/hydrogen-instructions.h (right):
> 
>
https://codereview.chromium.org/14761010/diff/1/src/hydrogen-instructions.h#n...
> src/hydrogen-instructions.h:2362: ElementsKind kind() const { return kind_; }
> There are so many "kind"s in V8 already, can we rename the field and the
> accessor to "elements_kind" here?