On Tue, 18 Oct 2022 22:51:51 GMT, Sandhya Viswanathan <sviswanat...@openjdk.org> wrote:
>> Volodymyr Paprotski has updated the pull request with a new target base due >> to a merge or a rebase. The pull request now contains 12 commits: >> >> - Merge remote-tracking branch 'origin/master' into avx512-poly >> - address Jamil's review >> - invalidkeyexception and some review comments >> - extra whitespace character >> - assembler checks and test case fixes >> - Merge remote-tracking branch 'origin/master' into avx512-poly >> - Merge remote-tracking branch 'origin' into avx512-poly >> - further restrict UsePolyIntrinsics with supports_avx512vlbw >> - missed white-space fix >> - - Fix whitespace and copyright statements >> - Add benchmark >> - ... and 2 more: https://git.openjdk.org/jdk/compare/9d3b4ef2...38d9e83c > > src/java.base/share/classes/com/sun/crypto/provider/Poly1305.java line 286: > >> 284: * numeric values. >> 285: */ >> 286: private void setRSVals() { //throws InvalidKeyException { > > The R and S check for invalid key (all bytes zero) could be submitted as a > separate PR. > It is not related to the Poly1305 acceleration. done, added a flag ------------- PR: https://git.openjdk.org/jdk/pull/10582
