[8u-dev, ppc] RFR for non-clean backport of 8185979: PPC64: Implement SHA2 intrinsic
Kazunori Ogata
OGATAK at jp.ibm.com
Thu Jun 6 08:45:51 UTC 2019
Hi Martin,
Thank you for your review.
Regards,
Ogata
"Doerr, Martin" <martin.doerr at sap.com> wrote on 2019/06/06 17:29:07:
> From: "Doerr, Martin" <martin.doerr at sap.com>
> To: Kazunori Ogata <OGATAK at jp.ibm.com>, "hotspot-compiler-
> dev at openjdk.java.net" <hotspot-compiler-dev at openjdk.java.net>, "jdk8u-
> dev at openjdk.java.net" <jdk8u-dev at openjdk.java.net>
> Date: 2019/06/06 17:29
> Subject: [EXTERNAL] RE: [8u-dev, ppc] RFR for non-clean backport of
> 8185979: PPC64: Implement SHA2 intrinsic
>
> Hi Ogata,
>
> looks correct.
>
> Best regards,
> Martin
>
>
> > -----Original Message-----
> > From: hotspot-compiler-dev <hotspot-compiler-dev-
> > bounces at openjdk.java.net> On Behalf Of Kazunori Ogata
> > Sent: Donnerstag, 6. Juni 2019 09:55
> > To: hotspot-compiler-dev at openjdk.java.net; jdk8u-dev at openjdk.java.net
> > Subject: [8u-dev, ppc] RFR for non-clean backport of 8185979: PPC64:
> > Implement SHA2 intrinsic
> >
> > Hi,
> >
> > May I get review of non-clean backport of 8185979: PPC64: Implement
SHA2
> > intrinsic?
> >
> > The original patch itself can be applied cleanly (besides difference
of
> > the source directory structure). However, in jdk8u, it also needs to
> > change the arguments of make_runtime_call() based on the
> > CCallingCenventionRequiresAsLongs flag. So I made additional changes
to
> > src/share/vm/opto/library_call.cpp and runtime.cpp.
> >
> > To separate the change in the original patch and the additional
changes, I
> > made webrev incrementally. webrev.02 below only contains the changes
by
> > the original patch, and webrev.03 contains all changes including the
> > additional ones. The difference between the two webrevs is the
changes in
> > library_call.cpp and runtime.cpp.
> >
> >
> > Original bug report:
> > https://bugs.openjdk.java.net/browse/JDK-8185979
> >
> > Webrev based on the original patch:
> > http://cr.openjdk.java.net/~horii/jdk8u_aes_be/8185979/webrev.02/
> >
> > Webrev of all changes:
> > http://cr.openjdk.java.net/~horii/jdk8u_aes_be/8185979/webrev.03/
> >
> >
> > I verified there is no degradation in jtreg (make test) results in
both
> > fastdebug and release builds.
> >
> >
> > Regards,
> > Ogata
>
>
More information about the jdk8u-dev
mailing list