Optimize typed-array length loading.

Optimize typed-array length loading.

R=dslomov@chromium.org

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

[Toon Verwaest, 2013-09-19 17:22:09]

PTAL

[Dmitry Lomov (no reviews), 2013-09-19 17:45:46]

https://codereview.chromium.org/23484060/diff/1/src/ic.cc
File src/ic.cc (right):

https://codereview.chromium.org/23484060/diff/1/src/ic.cc#newcode1380
src/ic.cc:1380: } else if (receiver->IsJSTypedArray()) {
Are you sure you this code path is hit? For typed arrays,
callbacks->IsAccessorPair() is true, so we will not get here. Or what am I
missing?

[Toon Verwaest, 2013-09-19 17:46:57]

You are right, I realized its not. And somehow when I move it up, the result is
wrong. ... trying to fix it up :)

[Toon Verwaest, 2013-09-19 18:02:18]

PTAL again.

[Dmitry Lomov (no reviews), 2013-09-19 18:07:24]

lgtm! Thanks for taking care of this.

[Dmitry Lomov (no reviews), 2013-09-19 18:18:03]

On 2013/09/19 18:07:24, Dmitry Lomov (chromium) wrote:
> lgtm! Thanks for taking care of this.

Good catch, patch set 3 still lgtm

[Dmitry Lomov (no reviews), 2013-09-19 18:18:06]

On 2013/09/19 18:07:24, Dmitry Lomov (chromium) wrote:
> lgtm! Thanks for taking care of this.

Good catch, patch set 3 still lgtm

[Toon Verwaest, 2013-09-19 18:19:35]

Committed patchset #3 manually as r16845 (presubmit successful).