[turbofan] Initial support for keyed access to fast JSArrays.

[turbofan] Initial support for keyed access to fast JSArrays.

This adds some initial support for keyed element access to fast,
non-holey JSArray objects.

Also renames PropertyAccessInfoFactory to AccessInfoFactory and
PropertyAccessMode to AccessMode.

R=jarin@chromium.org
BUG=v8:4470
LOG=n

Committed: https://crrev.com/1195b0e24de6d6c0a343d444fee68e9f63e022e7
Cr-Commit-Position: refs/heads/master@{#31717}

[Benedikt Meurer, 2015-11-02 13:16:23]

Hey Jaro,

Here's the initial version of the keyed element access.
Please take a look.

Thanks,
Benedikt

[Jarin, 2015-11-02 14:10:00]

lgtm. let's see how we can cope with monster graphs...

https://codereview.chromium.org/1418213010/diff/1/src/compiler/access-info.cc
File src/compiler/access-info.cc (right):

https://codereview.chromium.org/1418213010/diff/1/src/compiler/access-info.cc...
src/compiler/access-info.cc:25: // return map->IsJSObjectMap() &&
!map->has_dictionary_elements() &&
Remove the commented-out code and instead write in text what is missing/wrong
with the code above.

https://codereview.chromium.org/1418213010/diff/1/src/compiler/access-info.cc...
src/compiler/access-info.cc:32: // TODO(bmeurer): Do something about the number
stuff.
More details about the "number stuff"?

[Benedikt Meurer, 2015-11-02 17:25:17]

https://codereview.chromium.org/1418213010/diff/1/src/compiler/access-info.cc
File src/compiler/access-info.cc (right):

https://codereview.chromium.org/1418213010/diff/1/src/compiler/access-info.cc...
src/compiler/access-info.cc:25: // return map->IsJSObjectMap() &&
!map->has_dictionary_elements() &&
On 2015/11/02 14:10:00, Jarin wrote:
> Remove the commented-out code and instead write in text what is missing/wrong
> with the code above.

Done.

https://codereview.chromium.org/1418213010/diff/1/src/compiler/access-info.cc...
src/compiler/access-info.cc:32: // TODO(bmeurer): Do something about the number
stuff.
On 2015/11/02 14:10:00, Jarin wrote:
> More details about the "number stuff"?

Done.

[Benedikt Meurer, 2015-11-02 17:28:04]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2015-11-02 17:28:04]

The patchset sent to the CQ was uploaded after l-g-t-m from jarin@chromium.org
Link to the patchset: https://codereview.chromium.org/1418213010/#ps20001
(title: "Address comments")

[commit-bot: I haz the power, 2015-11-02 17:28:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1418213010/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1418213010/20001

[commit-bot: I haz the power, 2015-11-02 18:29:51]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-02 18:30:18]

Patchset 2 (id:??) landed as
https://crrev.com/1195b0e24de6d6c0a343d444fee68e9f63e022e7
Cr-Commit-Position: refs/heads/master@{#31717}