[interpreter] Use ForInFilter stub.

[interpreter] Use ForInFilter stub.

BUG=v8:4280
LOG=N

Committed: https://crrev.com/31eefa438ad2777de69010c37a6e99786c2b3fb3
Cr-Commit-Position: refs/heads/master@{#38100}

[oth, 2016-07-27 12:53:11]

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

[commit-bot: I haz the power, 2016-07-27 12:53:21]

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

[oth, 2016-07-27 12:55:17]

oth@chromium.org changed reviewers:
+ mythria@chromium.org

[oth, 2016-07-27 12:55:17]

Mythri, PTAL, moving to Camillo's call stub for ForInFilter. Thanks!

[mythria, 2016-07-27 12:56:24]

lgtm.

Thanks,
Mythri

[commit-bot: I haz the power, 2016-07-27 13:13:52]

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

[commit-bot: I haz the power, 2016-07-27 13:13:53]

Dry run: This issue passed the CQ dry run.

[oth, 2016-07-27 13:26:15]

The CQ bit was checked by oth@chromium.org

[commit-bot: I haz the power, 2016-07-27 13:26:23]

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-27 13:27:59]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-27 13:31:38]

Description was changed from

==========
[interpreter] Use ForInFilter stub.

BUG=v8:4280
LOG=N
==========

to

==========
[interpreter] Use ForInFilter stub.

BUG=v8:4280
LOG=N

Committed: https://crrev.com/31eefa438ad2777de69010c37a6e99786c2b3fb3
Cr-Commit-Position: refs/heads/master@{#38100}
==========

[commit-bot: I haz the power, 2016-07-27 13:31:40]

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

[rmcilroy, 2016-08-05 14:10:51]

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

[rmcilroy, 2016-08-05 14:10:51]

https://codereview.chromium.org/2187713004/diff/1/src/interpreter/interpreter.cc
File src/interpreter/interpreter.cc (right):

https://codereview.chromium.org/2187713004/diff/1/src/interpreter/interpreter...
src/interpreter/interpreter.cc:2045: Node* result = __ CallStub(callable,
context, key, receiver);
Any reason you didn't inline this TF stub into the bytecode handler?

[oth, 2016-08-08 08:16:19]

On 2016/08/05 14:10:51, rmcilroy wrote:
>
https://codereview.chromium.org/2187713004/diff/1/src/interpreter/interpreter.cc
> File src/interpreter/interpreter.cc (right):
> 
>
https://codereview.chromium.org/2187713004/diff/1/src/interpreter/interpreter...
> src/interpreter/interpreter.cc:2045: Node* result = __ CallStub(callable,
> context, key, receiver);
> Any reason you didn't inline this TF stub into the bytecode handler?

No. https://codereview.chromium.org/2220343002