[8u] RFR 8130150: Implement BigInteger.montgomeryMultiply intrinsic

Vladimir Kozlov vladimir.kozlov at oracle.com
Sat Feb 13 00:37:00 UTC 2016


Looks fine but you need to send this to jdk8u-dev at openjdk.java.net to 
get approval for backport.

Vladimir

On 2/12/16 10:31 AM, Vladimir Kempik wrote:
> Webrev for jdk part:
> http://cr.openjdk.java.net/~vkempik/8130150/webrev_jdk.00/
>
>
> On 12.02.2016 21:29, Vladimir Kempik wrote:
>> Hello
>>
>> 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.
>>
>> Bug:https://bugs.openjdk.java.net/browse/JDK-8130150
>> Webrev:http://cr.openjdk.java.net/~vkempik/8130150/webrev_hs.00/
>>
>> Thanks
>> -Vladimir
>


More information about the hotspot-compiler-dev mailing list