[8u-dev, ppc] RFR for (almost clean) backport of 8158232
OGATAK at jp.ibm.com
Tue May 14 06:02:49 UTC 2019
Thank you for the suggestion. I'll proceed to put the fix request comment
and tag in the original bug report.
Thank you too for offering to sponsor this change. I'll let you know when
"Gustavo Romero" <gromero at linux.vnet.ibm.com> wrote on 2019/05/14
> From: "Gustavo Romero" <gromero at linux.vnet.ibm.com>
> To: Kazunori Ogata/Japan/IBM at IBMJP,
hotspot-compiler-dev at openjdk.java.net,
> jdk8u-dev at openjdk.java.net
> Date: 2019/05/14 04:59
> Subject: Re: [8u-dev, ppc] RFR for (almost clean) backport of 8158232
> Hi Ogata,
> Thanks for the backport and for the webrev.
> I understand that offset adjustments in general, and particularly for
> backport, are not considered a change that needs to be reviewed again.
> That said, and although I'm not a Reviewer, I tested it against SPECjvm
> microbenchmarks for byte, int, and long and reviewed the change for
> It looks good.
> Please, provide a "Fix Request" comment to the original bug explaining
> the backport is low risk and affects PPC64-only, accordingly to  and
> Then please add the label "jdk8u-fix-request" to it.
> Once the approval to push is granted I'll sponsor the change.
> Thank you.
> Best regards,
>  https://wiki.openjdk.java.net/display/jdk8u/Main
>  http://openjdk.java.net/projects/jdk-updates/approval.html
> On 05/10/2019 03:55 AM, Kazunori Ogata wrote:
> > Sorry, I forgot to put the links to the bug report and the original
> > changeset Also forgot to mention that this changeset is needed to
> > backport AES intrinsics support  on ppc64 big-endian.
> > Bug report:
> > https://bugs.openjdk.java.net/browse/JDK-8158232
> > Original change set
> > http://hg.openjdk.java.net/jdk/jdk/rev/987528901b83
> > Webrev:
> > http://cr.openjdk.java.net/~horii/jdk8u_aes_be/8158232/webrev.02/
> > Refs:
> >  https://bugs.openjdk.java.net/browse/JDK-8188868
> > Regards,
> > Ogata
> > "hotspot-compiler-dev" <hotspot-compiler-dev-bounces at openjdk.java.net>
> > wrote on 2019/05/10 15:30:05:
> >> From: "Kazunori Ogata" <OGATAK at jp.ibm.com>
> >> To: hotspot-compiler-dev at openjdk.java.net, jdk8u-dev at openjdk.java.net
> >> Date: 2019/05/10 15:31
> >> Subject: [8u-dev, ppc] RFR for (almost clean) backport of 8158232
> >> Sent by: "hotspot-compiler-dev"
> > <hotspot-compiler-dev-bounces at openjdk.java.net>
> >> Hi,
> >> May I get review for backport of 8158232: PPC64: improve byte, int
> >> long array copy stubs by using VSX instructions?
> >> This changeset looks no conflict with the latest jdk8u-dev code, but
> >> patch command failed to apply it. It seems the patch command lost
> >> code regions to apply patches because stubGenerator_ppc.cpp has sets
> >> similar (but slightly different) functions.
> >> I created new webrev mainly to update line numbers in the patch file.
> >> verified I can build fastdebug and release builds and there was no
> >> degradation in "make test" results.
> >> http://cr.openjdk.java.net/~horii/jdk8u_aes_be/8158232/webrev.02/
> >> Regards,
> >> Ogata
More information about the hotspot-compiler-dev