RFR(M): 8248191: PPC: Implement Load/Store Vector with lxvl/stxvl in Power10

[joserz at linux.ibm.com]

Hello Martin,

Actually, there isn't xxswapd, my bad. In fact we usually need xxswapd to fix the vector lanes after lxvd2x but, if I understand it correctly, that order makes no difference in Hotspot.
Site note: GCC does a similar job when generating code at -O1 or higher and they're also avoiding lxvd2x on Power10.

Do you want me to resend the e-mail without mentioning xxswapd?

Thank you!!

Jose

On Thu, Jul 02, 2020 at 03:57:22PM +0000, Doerr, Martin wrote:
> Where do we save xxswapd instructions?
> I can't see it in the webrev.
> 
> Best regards,
> Martin
> 
> > -----Original Message-----
> > From: hotspot-compiler-dev <hotspot-compiler-dev-
> > retn at openjdk.java.net> On Behalf Of joserz at linux.ibm.com
> > Sent: Mittwoch, 1. Juli 2020 21:49
> > To: hotspot-compiler-dev at openjdk.java.net
> > Cc: Michihiro Horie <HORIE at jp.ibm.com>
> > Subject: RFR(M): 8248191: PPC: Implement Load/Store Vector with lxvl/stxvl
> > in Power10
> > 
> > This patch introduces two instructions lxvl/stvxl and replaces the current
> > lxvd2x/stxvd2x to load and store vectors. Like lxvd2x/stxvd2x, lxvl/stxvl can
> > access unaligned effective addresses with the advantage of *not* requiring
> > xxswapd after lxvd2x (or before stxvd2x) to correct the lanes in little-endian
> > mode.
> > 
> > Webrev: https://cr.openjdk.java.net/~mhorie/8248191/webrev.00/
> > Bug: https://bugs.openjdk.java.net/browse/JDK-8248191
> > 
> > Thanks for your review!
> > 
> > Jose R. Ziviani