[Interpreter] Fixes PushArgsAndConstruct builtin to not store any data outside esp.

[Interpreter] Fixes PushArgsAndConstruct builtin to not store any data outside esp.

In ia32 PushArgsAndConstruct builtin, we run out of registers and need to
temporarily store the data in the stack. In the earlier implementation,
a location outside the esp was used. This causes a problem if there is a
interrupt/signals which would use the same stack and corrupt the data that
is above the esp. This cl fixes it by pushing it onto the stack so that
the stack pointer is updated and hence the corruption will not happen. We
reuse the slot meant for receiver as a temporary store.

TBR=rmcilroy@chromium.org
BUG=v8:4280
LOG=N

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

[mythria, 2016-03-01 14:51:17]

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

[commit-bot: I haz the power, 2016-03-01 14:51:23]

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

[commit-bot: I haz the power, 2016-03-01 15:15:13]

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

[commit-bot: I haz the power, 2016-03-01 15:15:14]

Dry run: This issue passed the CQ dry run.

[mythria, 2016-03-01 15:16:58]

Description was changed from

==========
[Interpreter] Fixes PushArgsAndConstruct builtin to not store any data outside
esp.

In ia32 PushArgsAndConstruct builtin, we run out of registers and need to
temporarily store the data in the stack. In the earlier implementation,
a location outside the esp was used. This causes a problem if there is a
interrupt/signals which would use the same stack and corrupt the data that
is above the esp. This cl fixes it by pushing it onto the stack so that
the stack pointer is updated and hence the corruption will not happen. We
reuse the slot meant for receiver as a temporary store.

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

to

==========
[Interpreter] Fixes PushArgsAndConstruct builtin to not store any data outside
esp.

In ia32 PushArgsAndConstruct builtin, we run out of registers and need to
temporarily store the data in the stack. In the earlier implementation,
a location outside the esp was used. This causes a problem if there is a
interrupt/signals which would use the same stack and corrupt the data that
is above the esp. This cl fixes it by pushing it onto the stack so that
the stack pointer is updated and hence the corruption will not happen. We
reuse the slot meant for receiver as a temporary store.

TBR=rmcilroy@chromium.org
BUG=v8:4280
LOG=N
==========

[mythria, 2016-03-01 15:17:08]

The CQ bit was checked by mythria@chromium.org

[commit-bot: I haz the power, 2016-03-01 15:17:14]

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

[commit-bot: I haz the power, 2016-03-01 15:19:29]

Description was changed from

==========
[Interpreter] Fixes PushArgsAndConstruct builtin to not store any data outside
esp.

In ia32 PushArgsAndConstruct builtin, we run out of registers and need to
temporarily store the data in the stack. In the earlier implementation,
a location outside the esp was used. This causes a problem if there is a
interrupt/signals which would use the same stack and corrupt the data that
is above the esp. This cl fixes it by pushing it onto the stack so that
the stack pointer is updated and hence the corruption will not happen. We
reuse the slot meant for receiver as a temporary store.

TBR=rmcilroy@chromium.org
BUG=v8:4280
LOG=N
==========

to

==========
[Interpreter] Fixes PushArgsAndConstruct builtin to not store any data outside
esp.

In ia32 PushArgsAndConstruct builtin, we run out of registers and need to
temporarily store the data in the stack. In the earlier implementation,
a location outside the esp was used. This causes a problem if there is a
interrupt/signals which would use the same stack and corrupt the data that
is above the esp. This cl fixes it by pushing it onto the stack so that
the stack pointer is updated and hence the corruption will not happen. We
reuse the slot meant for receiver as a temporary store.

TBR=rmcilroy@chromium.org
BUG=v8:4280
LOG=N
==========

[commit-bot: I haz the power, 2016-03-01 15:19:30]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-01 15:20:14]

Description was changed from

==========
[Interpreter] Fixes PushArgsAndConstruct builtin to not store any data outside
esp.

In ia32 PushArgsAndConstruct builtin, we run out of registers and need to
temporarily store the data in the stack. In the earlier implementation,
a location outside the esp was used. This causes a problem if there is a
interrupt/signals which would use the same stack and corrupt the data that
is above the esp. This cl fixes it by pushing it onto the stack so that
the stack pointer is updated and hence the corruption will not happen. We
reuse the slot meant for receiver as a temporary store.

TBR=rmcilroy@chromium.org
BUG=v8:4280
LOG=N
==========

to

==========
[Interpreter] Fixes PushArgsAndConstruct builtin to not store any data outside
esp.

In ia32 PushArgsAndConstruct builtin, we run out of registers and need to
temporarily store the data in the stack. In the earlier implementation,
a location outside the esp was used. This causes a problem if there is a
interrupt/signals which would use the same stack and corrupt the data that
is above the esp. This cl fixes it by pushing it onto the stack so that
the stack pointer is updated and hence the corruption will not happen. We
reuse the slot meant for receiver as a temporary store.

TBR=rmcilroy@chromium.org
BUG=v8:4280
LOG=N

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

[commit-bot: I haz the power, 2016-03-01 15:20:15]

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