RFR: 8175811: [JVMCI] StubRoutines::_multiplyToLen symbol needs to exported

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

RFR: 8175811: [JVMCI] StubRoutines::_multiplyToLen symbol needs to exported

Doug Simon @ Oracle
Graal needs to get the address of StubRoutines::_multiplyToLen to intrinsify BigInteger.multiplyToLen. The export of this address was accidentally omitted.

http://cr.openjdk.java.net/~dnsimon/8175811/webrev/
https://bugs.openjdk.java.net/browse/JDK-8175811

-Doug
Reply | Threaded
Open this post in threaded view
|

Re: RFR: 8175811: [JVMCI] StubRoutines::_multiplyToLen symbol needs to exported

Tobias Hartmann-2
Hi,

On 24.02.2017 11:45, Doug Simon wrote:
> Graal needs to get the address of StubRoutines::_multiplyToLen to intrinsify BigInteger.multiplyToLen. The export of this address was accidentally omitted.
>
> http://cr.openjdk.java.net/~dnsimon/8175811/webrev/
> https://bugs.openjdk.java.net/browse/JDK-8175811

Looks good to me.

Best regards,
Tobias

>
> -Doug
>
Reply | Threaded
Open this post in threaded view
|

Re: RFR: 8175811: [JVMCI] StubRoutines::_multiplyToLen symbol needs to exported

Doug Simon @ Oracle
Thanks for the review.

-Doug

> On 24 Feb 2017, at 11:50, Tobias Hartmann <[hidden email]> wrote:
>
> Hi,
>
> On 24.02.2017 11:45, Doug Simon wrote:
>> Graal needs to get the address of StubRoutines::_multiplyToLen to intrinsify BigInteger.multiplyToLen. The export of this address was accidentally omitted.
>>
>> http://cr.openjdk.java.net/~dnsimon/8175811/webrev/
>> https://bugs.openjdk.java.net/browse/JDK-8175811
>
> Looks good to me.
>
> Best regards,
> Tobias
>
>>
>> -Doug
>>