PPC/s390: [Interpreter] Adds stackcheck in InterpreterPushArgsAndCall/Construct builtins.

PPC/s390: [Interpreter] Adds stackcheck in InterpreterPushArgsAndCall/Construct builtins.

    Port 7f3d15aad423aabf2f9116a929c8fd750615610a

    Original commit message:
    In ignition, arguments to function calls and function constructors are
    pushed onto the stack before calling the function. It is required to check
    that stack does not overflow when pushing the arguments.

    R=mythria@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com
BUG=

Committed: https://crrev.com/3b4cc88e9cec2fcbee0ca99514d876a6b30874b3
Cr-Commit-Position: refs/heads/master@{#39561}

[john.yan, 2016-09-20 16:55:29]

lgtm with comments.

https://codereview.chromium.org/2356583003/diff/1/src/builtins/s390/builtins-...
File src/builtins/s390/builtins-s390.cc (right):

https://codereview.chromium.org/2356583003/diff/1/src/builtins/s390/builtins-...
src/builtins/s390/builtins-s390.cc:2916: Generate_StackOverflowCheck(masm, r4,
r8, &stack_overflow);
r8 should be r7?

https://codereview.chromium.org/2356583003/diff/1/src/builtins/s390/builtins-...
src/builtins/s390/builtins-s390.cc:2954: Generate_StackOverflowCheck(masm, r4,
r8, &stack_overflow);
r8 should be r7?

[yangtian.zi, 2016-09-20 17:05:12]

The CQ bit was checked by ziyang@ca.ibm.com

[yangtian.zi, 2016-09-20 17:05:13]

The patchset sent to the CQ was uploaded after l-g-t-m from jyan@ca.ibm.com
Link to the patchset: https://codereview.chromium.org/2356583003/#ps20001
(title: "Applied suggested changes.")

[commit-bot: I haz the power, 2016-09-20 17:05:16]

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-20 17:52:39]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-20 17:53:06]

Description was changed from

==========
PPC/s390: [Interpreter] Adds stackcheck in InterpreterPushArgsAndCall/Construct
builtins.

    Port 7f3d15aad423aabf2f9116a929c8fd750615610a

    Original commit message:
    In ignition, arguments to function calls and function constructors are
    pushed onto the stack before calling the function. It is required to check
    that stack does not overflow when pushing the arguments.

    R=mythria@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com,
michael_dawson@ca.ibm.com, mbrandy@us.ibm.com
BUG=
==========

to

==========
PPC/s390: [Interpreter] Adds stackcheck in InterpreterPushArgsAndCall/Construct
builtins.

    Port 7f3d15aad423aabf2f9116a929c8fd750615610a

    Original commit message:
    In ignition, arguments to function calls and function constructors are
    pushed onto the stack before calling the function. It is required to check
    that stack does not overflow when pushing the arguments.

    R=mythria@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com,
michael_dawson@ca.ibm.com, mbrandy@us.ibm.com
BUG=

Committed: https://crrev.com/3b4cc88e9cec2fcbee0ca99514d876a6b30874b3
Cr-Commit-Position: refs/heads/master@{#39561}
==========

[commit-bot: I haz the power, 2016-09-20 17:53:07]

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

[yangtian.zi, 2016-10-05 21:25:54]

Patchset #3 (id:40001) has been deleted