[array] speed up array.forEach and friends by directly using in-operator

[array] speed up array.forEach and friends by directly using in-operator

The in-operator has recently been optimized making our smart macros obsolete.
This should give up to 10% speedup on forEach and friends.

BUG=v8:5041
Committed: https://crrev.com/eff24bef5c0f071008bdd4bcee3a86384e90c90b
Cr-Commit-Position: refs/heads/master@{#36576}

[Camillo Bruni, 2016-05-25 16:44:47]

Description was changed from

==========
[array] speed up array.forEach and friends by directly using in-operator

The in-operator has recently been optimized making our smart macros obsolete.

BUG=
==========

to

==========
[array] speed up array.forEach and friends by directly using in-operator

The in-operator has recently been optimized making our smart macros obsolete.

BUG=v8:5041
==========

[Camillo Bruni, 2016-05-25 16:45:24]

Description was changed from

==========
[array] speed up array.forEach and friends by directly using in-operator

The in-operator has recently been optimized making our smart macros obsolete.

BUG=v8:5041
==========

to

==========
[array] speed up array.forEach and friends by directly using in-operator

The in-operator has recently been optimized making our smart macros obsolete.
This should give an easy 10% speedup on forEach.

BUG=v8:5041
==========

[Camillo Bruni, 2016-05-25 16:45:25]

cbruni@chromium.org changed reviewers:
+ jkummerow@chromium.org

[Camillo Bruni, 2016-05-30 08:36:02]

Description was changed from

==========
[array] speed up array.forEach and friends by directly using in-operator

The in-operator has recently been optimized making our smart macros obsolete.
This should give an easy 10% speedup on forEach.

BUG=v8:5041
==========

to

==========
[array] speed up array.forEach and friends by directly using in-operator

The in-operator has recently been optimized making our smart macros obsolete.
This should give up to 10% speedup on forEach and friends.

BUG=v8:5041
==========

[Camillo Bruni, 2016-05-30 08:36:09]

PTAL

[Jakob Kummerow, 2016-05-30 08:41:02]

Oh, the simplification! LGTM.

[Camillo Bruni, 2016-05-30 09:36:22]

The CQ bit was checked by cbruni@chromium.org

[commit-bot: I haz the power, 2016-05-30 09:36:26]

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

[commit-bot: I haz the power, 2016-05-30 10:01:19]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-30 10:03:43]

Description was changed from

==========
[array] speed up array.forEach and friends by directly using in-operator

The in-operator has recently been optimized making our smart macros obsolete.
This should give up to 10% speedup on forEach and friends.

BUG=v8:5041
==========

to

==========
[array] speed up array.forEach and friends by directly using in-operator

The in-operator has recently been optimized making our smart macros obsolete.
This should give up to 10% speedup on forEach and friends.

BUG=v8:5041

Committed: https://crrev.com/eff24bef5c0f071008bdd4bcee3a86384e90c90b
Cr-Commit-Position: refs/heads/master@{#36576}
==========

[commit-bot: I haz the power, 2016-05-30 10:03:44]

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

[adamk, 2016-06-03 00:06:25]

adamk@chromium.org changed reviewers:
+ adamk@chromium.org

[adamk, 2016-06-03 00:06:25]

Thanks for the cleanup (I was always a bit embarrassed by this fast path). Maybe
we should just get rid of HAS_INDEX altogether (I'll send you a CL)?

[Camillo Bruni, 2016-06-03 08:16:08]

On 2016/06/03 at 00:06:25, adamk wrote:
> Thanks for the cleanup (I was always a bit embarrassed by this fast path).
Maybe we should just get rid of HAS_INDEX altogether (I'll send you a CL)?

yeah sounds good!