RFR [XXS] : Adjust the comment for flag UseAES in globals.hpp

Baesken, Matthias matthias.baesken at sap.com
Mon Feb 6 07:43:48 UTC 2017


Hi Vladimir, that's a good idea , thanks for adjusting it.

Best regards, Matthias


> -----Original Message-----
> From: Vladimir Kozlov [mailto:vladimir.kozlov at oracle.com]
> Sent: Freitag, 3. Februar 2017 18:47
> To: Baesken, Matthias <matthias.baesken at sap.com>; hotspot-
> dev at openjdk.java.net
> Cc: Langer, Christoph <christoph.langer at sap.com>
> Subject: Re: RFR [XXS] : Adjust the comment for flag UseAES in globals.hpp
> 
> Hi Matthias,
> 
> I looked on other flags and I want to remove line 667 too if you are
> okay with it:
> 
>   665   product(bool, UseSHA, false,                            \
>   666           "Control whether SHA instructions can be used " \
>   667           "on SPARC, on ARM and on x86")                  \
> 
> Thanks,
> Vladimir
> 
> On 2/3/17 3:52 AM, Baesken, Matthias wrote:
> >>>/Hi Matthias,/
> >
> >>>/ /
> >
> >>>/looks good./
> >
> >>
> >
> >>+1
> >
> >>
> >
> >>>/ /
> >
> >>>/I'm wondering whether this textual change could still make it into
> > JDK9...?/
> >
> >>
> >
> >>Make it P3 and change "Affects" and "Fix" version to 9.
> >
> >>
> >
> >> Thanks,
> >
> >> Vladimir
> >
> >
> >
> > Hi Vladimir, I adjusted the bug as suggested.
> >
> >
> >
> > Additional JDK9 webrev :
> >
> >
> >
> > http://cr.openjdk.java.net/~mbaesken/webrevs/8173825.1/
> >
> >
> >
> > Can you please sponsor it ?
> >
> >
> >
> > Thanks, Matthias
> >
> >
> >
> >
> >
> >>>/ /
> >
> >>>/Best regards/
> >
> >>>/Christoph/
> >
> >>>/ /
> >
> >>>/ /
> >
> >>>/From: Baesken, Matthias/
> >
> >>>/Sent: Donnerstag, 2. Februar 2017 15:44/
> >
> >>>/To: //hotspot-dev at openjdk.java.net
> > <http://mail.openjdk.java.net/mailman/listinfo/hotspot-dev>///
> >
> >>>/Cc: Schmidt, Lutz <//lutz.schmidt at sap.com
> > <http://mail.openjdk.java.net/mailman/listinfo/hotspot-dev>//>; Langer,
> Christoph <christoph.langer at sap.com
> > <http://mail.openjdk.java.net/mailman/listinfo/hotspot-dev>//>; Simonis,
> Volker <volker.simonis at sap.com
> > <http://mail.openjdk.java.net/mailman/listinfo/hotspot-dev>//>/
> >
> >>>/Subject: RFR [XXS] : Adjust the comment for flag UseAES in globals.hpp/
> >
> >>>/ /
> >
> >>>/Hello,/
> >
> >>>/     could I please have a  review for  the following small adjustment
> > to the comment of UseAES in globals.hpp  ./
> >
> >>>/UseAES is available now on more than x86/x64, for example  s390x and
> > ppc64le so the comment in globals.hpp needs to be adjusted./
> >
> >>>/I also need a sponsor for this./
> >
> >>>/ /
> >
> >>>/Bug :/
> >
> >>>/ /
> >
> >>>///https://bugs.openjdk.java.net/browse/JDK-8173825///
> >
> >>>/ /
> >
> >>>/Webrev :/
> >
> >>>/ /
> >
> >>>///http://cr.openjdk.java.net/~mbaesken/webrevs/8173825.0////
> >
> >>>/ /
> >
> >
> >


More information about the hotspot-dev mailing list