RFR: 8373059: Test sun/security/provider/acvp/ML_DSA_Intrinsic_Test.java should pass on Aarch64
Ferenc Rakoczi
duke at openjdk.org
Fri Dec 12 12:49:10 UTC 2025
On Tue, 9 Dec 2025 18:19:04 GMT, Volodymyr Paprotski <vpaprotski at openjdk.org> wrote:
>> …hould pass on Aarch64
>>
>> The test used to fail because it had checked a stronger equivalence of the results of the Java method and its intrinsified version.
>> Other then fixing that, I did some formatting and corrected a comment.
>
> test/jdk/sun/security/provider/pqc/ML_DSA_Intrinsic_Test.java line 147:
>
>> 145:
>> 146: if (!Arrays.equals(prod1, prod2)) {
>> 147: boolean modQequal = true;
>
> I would probably had moved this to its own helper `arraysCongruent` and replaces the `if (!Arrays.equals(prod1, prod2))` with `!arraysCongruent(prod1, prod2)`. But not a deal-breaker..
Thanks for your thorough review and comments!
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28722#discussion_r2614097633
More information about the security-dev
mailing list