[ic][ia32][x87] Don't push/pop value/slot/vector in store handlers.

[ic][ia32][x87] Don't push/pop value/slot/vector in store handlers.

According to new store IC calling convention the value, slot and vector are passed
on the stack and there's no need in trying to preserve values or respective registers
in store handlers.

Nice bonus: we also don't need virtual registers anymore.

BUG=v8:5407
Committed: https://crrev.com/49695346ae3f09dcbfe8ed127788a8827233cee0
Cr-Commit-Position: refs/heads/master@{#39672}

[Igor Sheludko, 2016-09-22 08:55:46]

Patchset #1 (id:1) has been deleted

[Igor Sheludko, 2016-09-22 09:01:07]

Description was changed from

==========
[ic][ia32][x87] Don't push/pop vector/slot that is already passed on stack.

... and don't use virtual registers anymore.

BUG=v8:5407
==========

to

==========
[ic][ia32][x87] Don't push/pop vector/slot in store handlers. 

According to new store IC calling convention vector and slot are passed on the
stack
and there's no need in trying to preserve the vector/slot registers' values in
handlers.

Nice bonus: we also don't need virtual registers anymore.

BUG=v8:5407
==========

[Igor Sheludko, 2016-09-22 09:01:13]

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

[commit-bot: I haz the power, 2016-09-22 09:01:24]

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

[Igor Sheludko, 2016-09-22 09:01:31]

ishell@chromium.org changed reviewers:
+ mvstanton@chromium.org

[Igor Sheludko, 2016-09-22 09:01:32]

PTAL

[Igor Sheludko, 2016-09-22 09:03:23]

Description was changed from

==========
[ic][ia32][x87] Don't push/pop vector/slot in store handlers. 

According to new store IC calling convention vector and slot are passed on the
stack
and there's no need in trying to preserve the vector/slot registers' values in
handlers.

Nice bonus: we also don't need virtual registers anymore.

BUG=v8:5407
==========

to

==========
[ic][ia32][x87] Don't push/pop value/slot/vector in store handlers. 

According to new store IC calling convention value, slot and vector are passed
on the stack and there's no need in trying to preserve values or respective
registers
in store handlers.

Nice bonus: we also don't need virtual registers anymore.

BUG=v8:5407
==========

[Igor Sheludko, 2016-09-22 09:04:58]

Description was changed from

==========
[ic][ia32][x87] Don't push/pop value/slot/vector in store handlers. 

According to new store IC calling convention value, slot and vector are passed
on the stack and there's no need in trying to preserve values or respective
registers
in store handlers.

Nice bonus: we also don't need virtual registers anymore.

BUG=v8:5407
==========

to

==========
[ic][ia32][x87] Don't push/pop value/slot/vector in store handlers. 

According to new store IC calling convention the value, slot and vector are
passed
on the stack and there's no need in trying to preserve values or respective
registers
in store handlers.

Nice bonus: we also don't need virtual registers anymore.

BUG=v8:5407
==========

[commit-bot: I haz the power, 2016-09-22 09:29:52]

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

[commit-bot: I haz the power, 2016-09-22 09:30:03]

Dry run: This issue passed the CQ dry run.

[mvstanton, 2016-09-23 11:14:36]

LGTM thanks for retiring the virtual register! :D

[Igor Sheludko, 2016-09-23 11:33:24]

The CQ bit was checked by ishell@chromium.org

[commit-bot: I haz the power, 2016-09-23 11:33:29]

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

[Igor Sheludko, 2016-09-23 11:52:33]

The CQ bit was unchecked by ishell@chromium.org

[Igor Sheludko, 2016-09-23 11:52:42]

The CQ bit was checked by ishell@chromium.org

[commit-bot: I haz the power, 2016-09-23 11:52:49]

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

[Igor Sheludko, 2016-09-23 13:03:52]

The CQ bit was unchecked by ishell@chromium.org

[Igor Sheludko, 2016-09-23 13:10:54]

The CQ bit was checked by ishell@chromium.org

[commit-bot: I haz the power, 2016-09-23 13:11:02]

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

[Igor Sheludko, 2016-09-23 13:19:14]

The CQ bit was unchecked by ishell@chromium.org

[Igor Sheludko, 2016-09-23 14:10:40]

The CQ bit was checked by ishell@chromium.org

[commit-bot: I haz the power, 2016-09-23 14:10:52]

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-09-23 14:29:12]

Description was changed from

==========
[ic][ia32][x87] Don't push/pop value/slot/vector in store handlers. 

According to new store IC calling convention the value, slot and vector are
passed
on the stack and there's no need in trying to preserve values or respective
registers
in store handlers.

Nice bonus: we also don't need virtual registers anymore.

BUG=v8:5407
==========

to

==========
[ic][ia32][x87] Don't push/pop value/slot/vector in store handlers. 

According to new store IC calling convention the value, slot and vector are
passed
on the stack and there's no need in trying to preserve values or respective
registers
in store handlers.

Nice bonus: we also don't need virtual registers anymore.

BUG=v8:5407
==========

[commit-bot: I haz the power, 2016-09-23 14:29:15]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-09-23 14:29:27]

Description was changed from

==========
[ic][ia32][x87] Don't push/pop value/slot/vector in store handlers. 

According to new store IC calling convention the value, slot and vector are
passed
on the stack and there's no need in trying to preserve values or respective
registers
in store handlers.

Nice bonus: we also don't need virtual registers anymore.

BUG=v8:5407
==========

to

==========
[ic][ia32][x87] Don't push/pop value/slot/vector in store handlers.

According to new store IC calling convention the value, slot and vector are
passed
on the stack and there's no need in trying to preserve values or respective
registers
in store handlers.

Nice bonus: we also don't need virtual registers anymore.

BUG=v8:5407

Committed: https://crrev.com/49695346ae3f09dcbfe8ed127788a8827233cee0
Cr-Commit-Position: refs/heads/master@{#39672}
==========

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

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