s390: [interpreter] Make the binary op with Smi bytecode handlers collect type feedback.

s390: [interpreter] Make the binary op with Smi bytecode handlers collect type feedback.

Port b305c7dfcbc11bf9d6f440cba039d733bade92c3

Original commit message:

    Drive-by fix: the order of parameters in the BinaryOpWithFeedback TurboFan code stubs now reflects the convention of having the context at the end.

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

BUG=v8:5273
LOG=N

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

[JaideepBajwa, 2016-08-23 15:54:13]

PTAL

[john.yan, 2016-08-23 16:31:55]

lgtm with one question.

https://codereview.chromium.org/2264353003/diff/1/src/s390/interface-descript...
File src/s390/interface-descriptors-s390.cc (right):

https://codereview.chromium.org/2264353003/diff/1/src/s390/interface-descript...
src/s390/interface-descriptors-s390.cc:276: Register registers[] = {r3, r2, r6,
r5};
was it a intent to use r3 first?

[JaideepBajwa, 2016-08-23 17:41:53]

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

[commit-bot: I haz the power, 2016-08-23 17:41:55]

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

[JaideepBajwa, 2016-08-23 17:46:34]

https://codereview.chromium.org/2264353003/diff/1/src/s390/interface-descript...
File src/s390/interface-descriptors-s390.cc (right):

https://codereview.chromium.org/2264353003/diff/1/src/s390/interface-descript...
src/s390/interface-descriptors-s390.cc:276: Register registers[] = {r3, r2, r6,
r5};
On 2016/08/23 16:31:55, john.yan wrote:
> was it a intent to use r3 first?
yes r3 should be first, the register array is passed as lhs then rhs

[commit-bot: I haz the power, 2016-08-23 18:40:09]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-23 18:40:39]

Description was changed from

==========
s390: [interpreter] Make the binary op with Smi bytecode handlers collect type
feedback.

Port b305c7dfcbc11bf9d6f440cba039d733bade92c3

Original commit message:

    Drive-by fix: the order of parameters in the BinaryOpWithFeedback TurboFan
code stubs now reflects the convention of having the context at the end.

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

BUG=v8:5273
LOG=N
==========

to

==========
s390: [interpreter] Make the binary op with Smi bytecode handlers collect type
feedback.

Port b305c7dfcbc11bf9d6f440cba039d733bade92c3

Original commit message:

    Drive-by fix: the order of parameters in the BinaryOpWithFeedback TurboFan
code stubs now reflects the convention of having the context at the end.

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

BUG=v8:5273
LOG=N

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

[commit-bot: I haz the power, 2016-08-23 18:40:40]

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