[riscv-port] RFR: 8280845: riscv: Intrinsify BigInteger.montgomerySquare

Sat Jan 29 04:07:36 UTC 2022

On Fri, 28 Jan 2022 09:22:09 GMT, Yanhong Zhu <yzhu at openjdk.org> wrote:

> Intrinsify BigInteger.montgomerySquare on RISCV.
> On unmatched borad,  performance of SPECjvm2008 crypto with intrinsic has improved 40%. 
> Hotspot/jdk tier1 passed on the unmatched board. And all jtreg tests have been tested on Qemu without new failures.
> 
> The details are as follows：
> Tested with option “UseMontgomerySquareIntrinsic”.
> <html xmlns:v="urn:schemas-microsoft-com:vml"
> xmlns:o="urn:schemas-microsoft-com:office:office"
> xmlns:x="urn:schemas-microsoft-com:office:excel"
> xmlns="http://www.w3.org/TR/REC-html40">
> 
> <head>
> 
> <meta name=ProgId content=Excel.Sheet>
> <meta name=Generator content="Microsoft Excel 15">
> <link id=Main-File rel=Main-File
> href="file:///C:\Users\Z00540~1\AppData\Local\Temp\msohtmlclip1\01\clip.htm">
> <link rel=File-List
> href="file:///C:\Users\Z00540~1\AppData\Local\Temp\msohtmlclip1\01\clip_filelist.xml">
> 
> 
> 
> </head>
> 
> <body link="#0563C1" vlink="#954F72">
> 
> 
> 
> TRUE(ops/m) | 1 | 2 | 3 | 4 | geomean |  
> -- | -- | -- | -- | -- | -- | --
> crypto.aes | 6.04 | 6.07 | 5.78 | 5.76 | 5.91 | 100.81%
> crypto.rsa | 171.75 | 172.01 | 172.03 | 172.07 | 171.96 | 239.34%
> crypto.signverify | 63.91 | 63.77 | 63.96 | 63.76 | 63.85 | 114.35%
> crypto | 40.48 | 40.52 | 39.92 | 39.84 | 40.19 | 140.25%
> FALSE(ops/m) |   |   |   |   |   |  
> crypto.aes | 5.75 | 5.75 | 6.26 | 5.71 | 5.86 |  
> crypto.rsa | 72.86 | 73.22 | 68.87 | 72.53 | 71.85 |  
> crypto.signverify | 55.75 | 55.55 | 55.76 | 56.29 | 55.84 |  
> crypto | 28.58 | 28.6 | 28.87 | 28.57 | 28.65 |  
> 
> 
> 
> </body>
> 
> </html>

Great numbers. Looks good.

-------------

Marked as reviewed by fyang (Lead).

PR: https://git.openjdk.java.net/riscv-port/pull/56