[Interpreter] Maintain the parent frame pointer after load

[Interpreter] Maintain the parent frame pointer after load

Maintain the parent frame pointer in a variable after loading it to avoid
having to reload it each time it is required.

BUG=v8:4280
Committed: https://crrev.com/361548ca4ca94663208f3354319f0b92812b1c54
Cr-Commit-Position: refs/heads/master@{#37247}

[rmcilroy, 2016-06-23 16:33:32]

Patchset #1 (id:1) has been deleted

[rmcilroy, 2016-06-23 16:33:36]

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

[commit-bot: I haz the power, 2016-06-23 16:33:46]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2085353005/20001

[rmcilroy, 2016-06-23 16:43:39]

Description was changed from

==========
NOT FOR REVIEW - Cache frame pointer
==========

to

==========
[Interpreter] Maintain the parent frame pointer after load

Maintain the parent frame pointer in a variable after loading it to avoid
having to reload it each time it is required.

BUG=v8:4280
==========

[commit-bot: I haz the power, 2016-06-23 17:01:25]

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

[commit-bot: I haz the power, 2016-06-23 17:01:26]

Dry run: This issue passed the CQ dry run.

[rmcilroy, 2016-06-23 17:19:15]

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

[rmcilroy, 2016-06-23 17:19:16]

Orion: seems to have a couple of percentage point improvement on Octane-noopt.
PTAL, thanks.

[oth, 2016-06-24 09:27:29]

lgtm. Nice set of perf bumps with this CL :-)

https://codereview.chromium.org/2085353005/diff/20001/src/interpreter/interpr...
File src/interpreter/interpreter-assembler.h (right):

https://codereview.chromium.org/2085353005/diff/20001/src/interpreter/interpr...
src/interpreter/interpreter-assembler.h:171: // Returns the frame point for the
interpreted frame of the function being
s/point/pointer/

[rmcilroy, 2016-06-24 09:45:54]

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

[commit-bot: I haz the power, 2016-06-24 09:45:58]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2085353005/20001

[rmcilroy, 2016-06-24 09:46:03]

The CQ bit was unchecked by rmcilroy@chromium.org

[rmcilroy, 2016-06-24 09:48:09]

The CQ bit was checked by rmcilroy@chromium.org

[rmcilroy, 2016-06-24 09:48:10]

The patchset sent to the CQ was uploaded after l-g-t-m from oth@chromium.org
Link to the patchset: https://codereview.chromium.org/2085353005/#ps40001
(title: "Fix typo")

[rmcilroy, 2016-06-24 09:48:14]

https://codereview.chromium.org/2085353005/diff/20001/src/interpreter/interpr...
File src/interpreter/interpreter-assembler.h (right):

https://codereview.chromium.org/2085353005/diff/20001/src/interpreter/interpr...
src/interpreter/interpreter-assembler.h:171: // Returns the frame point for the
interpreted frame of the function being
On 2016/06/24 09:27:29, oth wrote:
> s/point/pointer/

Done.

[commit-bot: I haz the power, 2016-06-24 09:48:18]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2085353005/40001

[commit-bot: I haz the power, 2016-06-24 10:14:12]

Description was changed from

==========
[Interpreter] Maintain the parent frame pointer after load

Maintain the parent frame pointer in a variable after loading it to avoid
having to reload it each time it is required.

BUG=v8:4280
==========

to

==========
[Interpreter] Maintain the parent frame pointer after load

Maintain the parent frame pointer in a variable after loading it to avoid
having to reload it each time it is required.

BUG=v8:4280
==========

[commit-bot: I haz the power, 2016-06-24 10:14:13]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-06-24 10:16:55]

Description was changed from

==========
[Interpreter] Maintain the parent frame pointer after load

Maintain the parent frame pointer in a variable after loading it to avoid
having to reload it each time it is required.

BUG=v8:4280
==========

to

==========
[Interpreter] Maintain the parent frame pointer after load

Maintain the parent frame pointer in a variable after loading it to avoid
having to reload it each time it is required.

BUG=v8:4280

Committed: https://crrev.com/361548ca4ca94663208f3354319f0b92812b1c54
Cr-Commit-Position: refs/heads/master@{#37247}
==========

[commit-bot: I haz the power, 2016-06-24 10:16:56]

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