> With `quadKeccak`, it is possible to have 3 extra `keccak`s calls that are 
> really noops.. teach `SHA3Parallel.squeeze()` how many operations it really 
> should be doing.
> 
> While it is possible to match precisely the number of keccak calls required.. 
> the previous implementation always called doubleKeccak, so collapsing the odd 
> numbers to the closest even.
> 
> PS:  `make install-hsdis test 
> TEST="micro:org.openjdk.bench.javax.crypto.full.SignatureBench.MLDSA" 
> MICRO="JAVA_OPTIONS=-XX:+UnlockDiagnosticVMOptions 
> -XX:-UseSHA3Intrinsics;FORK=1;ITER=3;TIME=10;WARMUP_ITER=7;WARMUP_TIME=10;OPTIONS=-prof
>  perfasm -p algorithm=ML-DSA-65"`
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Volodymyr Paprotski has updated the pull request incrementally with one 
additional commit since the last revision:

  comment from Ferenc

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/31648/files
  - new: https://git.openjdk.org/jdk/pull/31648/files/03ead216..14f8acfe

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=31648&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=31648&range=00-01

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/31648.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/31648/head:pull/31648

PR: https://git.openjdk.org/jdk/pull/31648

Reply via email to