[turbofan] Refactor the lowering of element/property accesses.

[turbofan] Refactor the lowering of element/property accesses.

Split the monster methods in JSNativeContextSpecialization into smaller
ones, adding appropriate helpers. Improve the condition checking for
strings and numbers using CheckString/CheckNumber when applicable. Also
try to merge compatible PropertyAccessInfos, to avoid running into the
polymorphic case whenever possible.

Drive-by-fix: Don't try to resurrect dead nodes during LoadElimination.
With the improve code generation for monomorphic loads, we seem to
trigger the dead node resurrection.

R=epertoso@chromium.org
BUG=v8:4930, v8:5141
Committed: https://crrev.com/1bce27906d5933ad05310987e3dd91782324e712
Cr-Commit-Position: refs/heads/master@{#38127}

[Benedikt Meurer, 2016-07-28 09:08:42]

The CQ bit was checked by bmeurer@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-28 09:08:46]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Benedikt Meurer, 2016-07-28 09:09:58]

Hey Enrico,

Here's some refactoring in preparation for proper map check elimination.
Please take a look.

Thanks,
Benedikt

[commit-bot: I haz the power, 2016-07-28 09:30:57]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-28 09:30:58]

Dry run: This issue passed the CQ dry run.

[epertoso, 2016-07-28 09:51:31]

lgtm

[Benedikt Meurer, 2016-07-28 09:52:24]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-07-28 09:52:34]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-28 09:54:15]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-28 09:57:08]

Description was changed from

==========
[turbofan] Refactor the lowering of element/property accesses.

Split the monster methods in JSNativeContextSpecialization into smaller
ones, adding appropriate helpers. Improve the condition checking for
strings and numbers using CheckString/CheckNumber when applicable. Also
try to merge compatible PropertyAccessInfos, to avoid running into the
polymorphic case whenever possible.

Drive-by-fix: Don't try to resurrect dead nodes during LoadElimination.
With the improve code generation for monomorphic loads, we seem to
trigger the dead node resurrection.

R=epertoso@chromium.org
BUG=v8:4930,v8:5141
==========

to

==========
[turbofan] Refactor the lowering of element/property accesses.

Split the monster methods in JSNativeContextSpecialization into smaller
ones, adding appropriate helpers. Improve the condition checking for
strings and numbers using CheckString/CheckNumber when applicable. Also
try to merge compatible PropertyAccessInfos, to avoid running into the
polymorphic case whenever possible.

Drive-by-fix: Don't try to resurrect dead nodes during LoadElimination.
With the improve code generation for monomorphic loads, we seem to
trigger the dead node resurrection.

R=epertoso@chromium.org
BUG=v8:4930,v8:5141

Committed: https://crrev.com/1bce27906d5933ad05310987e3dd91782324e712
Cr-Commit-Position: refs/heads/master@{#38127}
==========

[commit-bot: I haz the power, 2016-07-28 09:57:09]

Patchset 1 (id:??) landed as
https://crrev.com/1bce27906d5933ad05310987e3dd91782324e712
Cr-Commit-Position: refs/heads/master@{#38127}