RFR(S): 8210660: PPC64: Mapping floating point registers to vsx registers in ppc.ad

[Doerr, Martin]

Hi Michihiro,

your webrev
http://cr.openjdk.java.net/~mhorie/8210660/webrev.01/
looks good to me.

I only noticed that some instructs (e.g. for xscvdpspn and xxspltw) have no "format %{ ... %}" specification so they are missing in the PrintOptoAssembly output. But this seems to be missing in the current version already.

We can test it while waiting for a 2nd review.

Thanks and best regards,
Martin


From: Michihiro Horie <HORIE at jp.ibm.com>
Sent: Donnerstag, 13. September 2018 19:05
To: Doerr, Martin <martin.doerr at sap.com>
Cc: Lindenmaier, Goetz <goetz.lindenmaier at sap.com>; Gustavo Romero <gromero at linux.vnet.ibm.com>; hotspot-compiler-dev at openjdk.java.net
Subject: RE: RFR(S): 8210660: PPC64: Mapping floating point registers to vsx registers in ppc.ad


Hi Martin,

Thank you so much for your review (and adding the ID in the subject :-).

>I don't think we need 2 nodes for ReplicateF with vector length 4. Both ones in your webrev are for Power8 so only one will be used.
You're right, thanks. I removed a redundant one.

I also refactored ReplicateD with vector length 2.

Following is the latest webrev:
http://cr.openjdk.java.net/~mhorie/8210660/webrev.01/<http://cr.openjdk.java.net/~mhorie/8210660/webrev.00/>


Best regards,
--
Michihiro,
IBM Research - Tokyo

[Inactive hide details for "Doerr, Martin" ---2018/09/13 16:25:33---Hi Michihiro, I have added "RFR(S): 8210660" to the subject.]"Doerr, Martin" ---2018/09/13 16:25:33---Hi Michihiro, I have added "RFR(S): 8210660" to the subject.

From: "Doerr, Martin" <martin.doerr at sap.com<mailto:martin.doerr at sap.com>>
To: Michihiro Horie <HORIE at jp.ibm.com<mailto:HORIE at jp.ibm.com>>, "hotspot-compiler-dev at openjdk.java.net<mailto:hotspot-compiler-dev at openjdk.java.net>" <hotspot-compiler-dev at openjdk.java.net<mailto:hotspot-compiler-dev at openjdk.java.net>>
Cc: Gustavo Romero <gromero at linux.vnet.ibm.com<mailto:gromero at linux.vnet.ibm.com>>, "Lindenmaier, Goetz" <goetz.lindenmaier at sap.com<mailto:goetz.lindenmaier at sap.com>>
Date: 2018/09/13 16:25
Subject: RE: RFR(S): 8210660: PPC64: Mapping floating point registers to vsx registers in ppc.ad

________________________________



Hi Michihiro,

I have added "RFR(S): 8210660" to the subject.

I don't think we need 2 nodes for ReplicateF with vector length 4. Both ones in your webrev are for Power8 so only one will be used.
Besides this, your change looks good to me.

Would you like to improve ReplicateD with vector length 2, too?

Thanks and best regards,
Martin


From: Michihiro Horie <HORIE at jp.ibm.com<mailto:HORIE at jp.ibm.com>>
Sent: Mittwoch, 12. September 2018 18:11
To: hotspot-compiler-dev at openjdk.java.net<mailto:hotspot-compiler-dev at openjdk.java.net>
Cc: Doerr, Martin <martin.doerr at sap.com<mailto:martin.doerr at sap.com>>; Gustavo Romero <gromero at linux.vnet.ibm.com<mailto:gromero at linux.vnet.ibm.com>>; Lindenmaier, Goetz <goetz.lindenmaier at sap.com<mailto:goetz.lindenmaier at sap.com>>
Subject: RFR: PPC64: Mapping floating point registers to vsx registers in ppc.ad

Dear all,

Would you please review the following change?

Bug: https://bugs.openjdk.java.net/browse/JDK-8210660
Webrev: http://cr.openjdk.java.net/~mhorie/8210660/webrev.00/

In the current code emit for replicating the floating point value in ppc.ad, a floating point value is once stored in order to load as an integer value. However, when SuperwordUseVSX is enabled, this is redundant because the floating point registers are overlapped with vsx registers 0-31. We can use vsx instructions for replicating the floating point value by mapping the floating point register to the vsx register.


Best regards,
--
Michihiro,
IBM Research - Tokyo


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20180914/a52251ac/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: image001.gif
Type: image/gif
Size: 105 bytes
Desc: image001.gif
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20180914/a52251ac/image001.gif>