s390: use new mul instruction

s390: use new mul instruction

R=joransiu@ca.ibm.com, bjaideep@ca.ibm.com, danno@chromium.org, bmeurer@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2691893002
Cr-Commit-Position: refs/heads/master@{#43166}
Committed: https://chromium.googlesource.com/v8/v8/+/2dab40ccda08c6382ae69e5d51bbc1c5c4f3a7aa

[john.yan, 2017-02-13 05:03:33]

ptal

[JoranSiu, 2017-02-13 14:43:56]

lgtm with a minor nit.

https://codereview.chromium.org/2691893002/diff/1/src/full-codegen/s390/full-...
File src/full-codegen/s390/full-codegen-s390.cc (right):

https://codereview.chromium.org/2691893002/diff/1/src/full-codegen/s390/full-...
src/full-codegen/s390/full-codegen-s390.cc:1599: // the result are identical.
nit. Collapse the last 2 lines of comments into 1?

https://codereview.chromium.org/2691893002/diff/1/src/full-codegen/s390/full-...
src/full-codegen/s390/full-codegen-s390.cc:1610: // the result are identical.
nit.  ditto as above.

[john.yan, 2017-02-13 14:50:46]

On 2017/02/13 14:43:56, JoranSiu wrote:
> lgtm with a minor nit.
> 
>
https://codereview.chromium.org/2691893002/diff/1/src/full-codegen/s390/full-...
> File src/full-codegen/s390/full-codegen-s390.cc (right):
> 
>
https://codereview.chromium.org/2691893002/diff/1/src/full-codegen/s390/full-...
> src/full-codegen/s390/full-codegen-s390.cc:1599: // the result are identical.
> nit. Collapse the last 2 lines of comments into 1?
> 
>
https://codereview.chromium.org/2691893002/diff/1/src/full-codegen/s390/full-...
> src/full-codegen/s390/full-codegen-s390.cc:1610: // the result are identical.
> nit.  ditto as above.

ok, it's the result of cl format. lol.

[john.yan, 2017-02-13 15:09:13]

The CQ bit was checked by jyan@ca.ibm.com to run a CQ dry run

[commit-bot: I haz the power, 2017-02-13 15:09:22]

Dry run: 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, 2017-02-13 15:37:19]

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

[commit-bot: I haz the power, 2017-02-13 15:37:20]

Dry run: This issue passed the CQ dry run.

[Benedikt Meurer, 2017-02-13 17:41:16]

Sure. LGTM!

[john.yan, 2017-02-13 17:54:15]

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

[john.yan, 2017-02-13 17:54:15]

The patchset sent to the CQ was uploaded after l-g-t-m from joransiu@ca.ibm.com
Link to the patchset: https://codereview.chromium.org/2691893002/#ps20001
(title: "fix comment")

[commit-bot: I haz the power, 2017-02-13 17:54:23]

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, 2017-02-13 18:50:36]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487008455149750,
"parent_rev": "d891b500535b1833d03f25e878bffede56ad3abc", "commit_rev":
"2dab40ccda08c6382ae69e5d51bbc1c5c4f3a7aa"}

[commit-bot: I haz the power, 2017-02-13 18:50:41]

Description was changed from

==========
s390: use new mul instruction

R=joransiu@ca.ibm.com, bjaideep@ca.ibm.com, danno@chromium.org,
bmeurer@chromium.org
BUG=
==========

to

==========
s390: use new mul instruction

R=joransiu@ca.ibm.com, bjaideep@ca.ibm.com, danno@chromium.org,
bmeurer@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2691893002
Cr-Commit-Position: refs/heads/master@{#43166}
Committed:
https://chromium.googlesource.com/v8/v8/+/2dab40ccda08c6382ae69e5d51bbc1c5c4f...
==========

[commit-bot: I haz the power, 2017-02-13 18:50:42]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/v8/v8/+/2dab40ccda08c6382ae69e5d51bbc1c5c4f...