[8u-dev] RFA 8130150: Implement BigInteger.montgomeryMultiply intrinsic
Vladimir Kempik
vladimir.kempik at oracle.com
Fri Feb 26 16:04:56 UTC 2016
Hello Sean
I have filled such bug yet because I haven't pushed the changeset yet,
so the "bug" isn't present at the moment in any of java repos.
Vladimir.
On 26.02.2016 18:31, Seán Coffey wrote:
> Vladimir Kozlov, Vladimir Kempik,
>
>> Separate bug is filed to resolve this issue later.
>
> can one of you give the bug ID capturing this ? Please link it to
> JDK-8130150
>
> Regards,
> Sean.
>
> On 19/02/16 18:24, Vladimir Kozlov wrote:
>> Changes are reviewed. There is difference from jdk9 because of issue
>> with code generated by SunStudio C++ version used to build jdk 8u and
>> 7u. As result next changes where made:
>>
>> + //montgomery_square fails to pass BigIntegerTest on solaris amd64
>> + //on jdk7 and jdk8.
>> +#ifndef SOLARIS
>> if (len >= MONTGOMERY_SQUARING_THRESHOLD) {
>> +#else
>> + if (0) {
>> +#endif
>> ::montgomery_square(a, n, m, (unsigned long)inv, longwords);
>>
>> Note, this change is fine since it affects only performance for some
>> range of values.
>> Separate bug is filed to resolve this issue later.
>>
>> Thanks,
>> Vladimir
>>
>> On 2/19/16 9:29 AM, Vladimir Kempik wrote:
>>> Hello
>>>
>>> I'd like to port this fix into 8u-dev.
>>>
>>> Implement BigInteger.montgomeryMultiply intrinsic
>>>
>>> The patch has two parts, for jdk and hotspot.
>>>
>>> Testing: jprt, testcase.
>>>
>>> Bug:https://bugs.openjdk.java.net/browse/JDK-8130150
>>> Webrev for
>>> hotspot:http://cr.openjdk.java.net/~vkempik/8130150/webrev_hs.01/
>>> Webrev for jdk:
>>> http://cr.openjdk.java.net/~vkempik/8130150/webrev_jdk.00/
>>> Review thread:
>>> http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2016-February/021321.html
>>>
>>> Thanks
>>> -Vladimir
>>>
>
More information about the jdk8u-dev
mailing list