[8u] RFR 8130150: Implement BigInteger.montgomeryMultiply intrinsic
volker.simonis at gmail.com
Wed Feb 17 10:50:04 UTC 2016
I think adding a public method to MutableBigInteger is fine because
MutableBigInteger is package private. So adding methods to it wouldn't
change the public API.
On Wed, Feb 17, 2016 at 11:10 AM, Doerr, Martin <martin.doerr at sap.com> wrote:
> is it allowed to add a public method to MutableBigInteger in jdk8u?
> I guess this will need additional approval. Alternatively, the method "static long inverseMod64(long val)" could be moved into BigInteger.java as private method.
> Otherwise, the jdk8u change looks good.
> Best regards,
> -----Original Message-----
> From: hotspot-compiler-dev [mailto:hotspot-compiler-dev-bounces at openjdk.java.net] On Behalf Of Vladimir Kozlov
> Sent: Samstag, 13. Februar 2016 01:37
> To: Vladimir Kempik <vladimir.kempik at oracle.com>; hotspot-compiler-dev at openjdk.java.net
> Subject: Re: [8u] RFR 8130150: Implement BigInteger.montgomeryMultiply intrinsic
> Looks fine but you need to send this to jdk8u-dev at openjdk.java.net to
> get approval for backport.
> On 2/12/16 10:31 AM, Vladimir Kempik wrote:
>> Webrev for jdk part:
>> On 12.02.2016 21:29, Vladimir Kempik wrote:
>>> Please review this backport of 8130150 to jdk8u.
>>> Implement BigInteger.montgomeryMultiply intrinsic
>>> The patch didn't apply cleanly, some changes were needed. Most of them are just to make patch apply, except one.
>>> for MACC and MACC2 defines, __asm__ were changed to asm, to make solaris compiler (SS12U1) happy.
>>> The patch has two parts, for jdk and hotspot.
>>> Testing: jprt, testcase.
More information about the hotspot-compiler-dev