X87: [Interpreter] Changes GenerateDoubleToObject to push and pop rsi value.

X87: [Interpreter] Changes GenerateDoubleToObject to push and pop rsi value.

  port e6b6e5545386985f1d0bec875307e5df52062d71 (r35163)

  original commit message:
  In the earlier implementation of GenerateDoubleToObject the context
  is loaded from the parent's frame. rsi is clobbered because it is used
  to store kHoleNan constnat. It is not always safe to peek at
  the parents frame. Bytecode handlers have TypedFrame and the type of
  frame is stored at FP + 1. GenerateDoubleToObject expects context
  to be store at that place. In the current implementation rsi is pushed
  onto the stack and is popped when exiting this function.

BUG=

Committed: https://crrev.com/a0d2f240c3efdb4c56bbdee4fc4cb40c1015b3d8
Cr-Commit-Position: refs/heads/master@{#35478}

[zhengxing.li, 2016-04-14 07:43:07]

zhengxing.li@intel.com changed reviewers:
+ chunyang.dai@intel.com, mythria@chromium.org, weiliang.lin@intel.com

[zhengxing.li, 2016-04-14 07:43:08]

PTAL, thanks!

[Weiliang, 2016-04-14 08:23:40]

lgtm

[mythria, 2016-04-14 08:57:38]

lgtm.

Thanks,
Mythri

[zhengxing.li, 2016-04-14 09:07:22]

The CQ bit was checked by zhengxing.li@intel.com

[commit-bot: I haz the power, 2016-04-14 09:07:31]

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

[commit-bot: I haz the power, 2016-04-14 09:28:09]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-14 09:28:47]

Description was changed from

==========
X87: [Interpreter] Changes GenerateDoubleToObject to push and pop rsi value.

  port e6b6e5545386985f1d0bec875307e5df52062d71 (r35163)

  original commit message:
  In the earlier implementation of GenerateDoubleToObject the context
  is loaded from the parent's frame. rsi is clobbered because it is used
  to store kHoleNan constnat. It is not always safe to peek at
  the parents frame. Bytecode handlers have TypedFrame and the type of
  frame is stored at FP + 1. GenerateDoubleToObject expects context
  to be store at that place. In the current implementation rsi is pushed
  onto the stack and is popped when exiting this function.

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

to

==========
X87: [Interpreter] Changes GenerateDoubleToObject to push and pop rsi value.

  port e6b6e5545386985f1d0bec875307e5df52062d71 (r35163)

  original commit message:
  In the earlier implementation of GenerateDoubleToObject the context
  is loaded from the parent's frame. rsi is clobbered because it is used
  to store kHoleNan constnat. It is not always safe to peek at
  the parents frame. Bytecode handlers have TypedFrame and the type of
  frame is stored at FP + 1. GenerateDoubleToObject expects context
  to be store at that place. In the current implementation rsi is pushed
  onto the stack and is popped when exiting this function.

BUG=

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

[commit-bot: I haz the power, 2016-04-14 09:28:48]

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