[riscv-port] RFR: 8278847: riscv: Intrinsify BigInteger.montgomeryMultiply

Fri Jan 28 02:30:41 UTC 2022

On Thu, 27 Jan 2022 03:47:11 GMT, Yanhong Zhu <yzhu at openjdk.org> wrote:

> Intrinsify BigInteger.montgomeryMultiply on RISCV.
> On unmatched borad, performance of SPECjvm2008 crypto has improved by 4%，of which sub-testcase crypto.rsa has improved by 7%.The details are as follows：
> 
> <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 | geomean | percent(true/false)
> -- | -- | -- | -- | -- | --
> crypto.aes | 6.15 | 6.26 | 5.82 | 6.07 | 102.81%
> crypto.rsa | 73.07 | 72.77 | 69.5 | 71.76 | 107.22%
> crypto.signverify | 56.82 | 57.13 | 56.34 | 56.76 | 102.59%
> crypto | 29.44 | 29.63 | 28.35 | 29.13 | 104.18%
> **FALSE(ops/m)** |   |   |   |   | 
> crypto.aes | 5.77 | 6.14 | 5.82 | 5.91 |  
> crypto.rsa | 67.09 | 67.26 | 66.44 | 66.93 |  
> crypto.signverify | 55.51 | 55.6 | 54.88 | 55.33 |  
> crypto | 27.8 | 28.42 | 27.68 | 27.96 |  
> 
> 
> 
> </body>
> 
> </html>
> All jtregs were tested on QEMU without new failures.

Looks good. This implementation has been there in our JDK distribution for a long time.

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

Marked as reviewed by fyang (Lead).

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