[turbofan] Extend Array.prototype.pop lowering to support FAST_DOUBLE_ELEMENTS.

[turbofan] Extend Array.prototype.pop lowering to support FAST_DOUBLE_ELEMENTS.

So far the Array.prototype.pop lowering in the JSBuiltinReducer was
limited to (holey) fast or fast-smi elements. But it can be made to
work easily to also handle fast-double elements, so allow that as
well.

R=jarin@chromium.org
BUG=v8:5267, v8:6338
Review-Url: https://codereview.chromium.org/2861443006
Cr-Commit-Position: refs/heads/master@{#45090}
Committed: https://chromium.googlesource.com/v8/v8/+/5c433ec500e0de20737624bceb0ade69d67a9153

[Benedikt Meurer, 2017-05-04 11:38:24]

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

[commit-bot: I haz the power, 2017-05-04 11:38:27]

Dry run: CQ is trying da patch.

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

[Jarin, 2017-05-04 11:45:19]

lgtm

[Benedikt Meurer, 2017-05-04 11:48:59]

The CQ bit was unchecked by bmeurer@chromium.org

[Benedikt Meurer, 2017-05-04 11:49:04]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2017-05-04 11:49:08]

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, 2017-05-04 12:35:58]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493898544727610, "parent_rev":
"418926e0104b2b634f38876b5b274840decf8e46", "commit_rev":
"5c433ec500e0de20737624bceb0ade69d67a9153"}

[commit-bot: I haz the power, 2017-05-04 12:36:04]

Description was changed from

==========
[turbofan] Extend Array.prototype.pop lowering to support FAST_DOUBLE_ELEMENTS.

So far the Array.prototype.pop lowering in the JSBuiltinReducer was
limited to (holey) fast or fast-smi elements. But it can be made to
work easily to also handle fast-double elements, so allow that as
well.

R=jarin@chromium.org
BUG=v8:5267,v8:6338
==========

to

==========
[turbofan] Extend Array.prototype.pop lowering to support FAST_DOUBLE_ELEMENTS.

So far the Array.prototype.pop lowering in the JSBuiltinReducer was
limited to (holey) fast or fast-smi elements. But it can be made to
work easily to also handle fast-double elements, so allow that as
well.

R=jarin@chromium.org
BUG=v8:5267,v8:6338

Review-Url: https://codereview.chromium.org/2861443006
Cr-Commit-Position: refs/heads/master@{#45090}
Committed:
https://chromium.googlesource.com/v8/v8/+/5c433ec500e0de20737624bceb0ade69d67...
==========

[commit-bot: I haz the power, 2017-05-04 12:36:05]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/5c433ec500e0de20737624bceb0ade69d67...