RFR(S): JDK-8191339: [JVMCI] BigInteger compiler intrinsics on Graal.

Vladimir Kozlov vladimir.kozlov at oracle.com
Fri Jun 22 16:04:45 UTC 2018


Hi Patric,

Do you need Graal changes for this? Or it already has these intrinsics 
and the only problem is these flags were not set in vm_version_x86.cpp?

Small note. In vm_version_x86.cpp previous code has already 
COMPILER2_OR_JVMCI check. You can remove previous #endif and new #ifdef. 
Also change comment for closing #endif at line 1080 to // COMPILER2_OR_JVMCI

1080 #endif // COMPILER2

What testing you did?

Thanks,
Vladimir

On 6/21/18 8:26 AM, Patric Hedlin wrote:
> Dear all,
> 
> I would like to ask for help to review the following change/update:
> 
> Issue:  https://bugs.openjdk.java.net/browse/JDK-8191339
> 
> Webrev: http://cr.openjdk.java.net/~phedlin/tr8191339/
> 
> 
> 8191339: [JVMCI] BigInteger compiler intrinsics on Graal.
> 
>      Enabling BigInteger intrinsics via JVMCI.
> 
> 
> 
> Best regards,
> Patric


More information about the hotspot-compiler-dev mailing list