JDK10/RFR(L): 8167199: Add C2 SPARC intrinsic for BigInteger::multiplyToLen() method.

Nils Eliasson nils.eliasson at oracle.com
Tue Aug 29 09:22:04 UTC 2017


Hi Patric,

Looks good.

Just make sure this code path is covered in the regular testing.

Regards,

Nils


On 2017-08-28 15:06, 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-8167199
>
> Webrev:  http://cr.openjdk.java.net/~phedlin/tr8167199/
>
>
> 8167199: Add C2 SPARC intrinsic for BigInteger::multiplyToLen() method.
>
>     Added C2 SPARC support for intrinsic 'multiplyToLen', including
>     generalised version based on 'mpmul' instruction (when available).
>     The code emitted should be subject for evaluation (e.g. MME on
>     SPARC Core C5 might be used in an improved version).
>
> Testing:
>
>     Mostly tested on JDK9 (jtreg/RBT/hotspot/tier0-comp).
>     Testing on JDK10 (jtreg/RBT/hotspot/precheckin-comp).
>
>
> Best regards,
> Patric



More information about the hotspot-compiler-dev mailing list