Skip to content

8351034: Add AVX-512 intrinsics for ML-DSA #23860

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 15 commits into from
Closed

8351034: Add AVX-512 intrinsics for ML-DSA #23860

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
1ff5851
JDK-8351034 Add AVX-512 intrinsics for ML-DSA
ferakocz Jan 10, 2025
fe50e0d
Added comments, removed debugging printfs
ferakocz Mar 3, 2025
331f1ec
Merged master.
ferakocz Mar 5, 2025
3aaa106
Added alignment to loop entries.
ferakocz Mar 5, 2025
64135f2
Accepted review comments.
ferakocz Mar 6, 2025
f65ef7c
Added validity test for the intrinsics.
ferakocz Mar 12, 2025
aa2fdf2
Made the intrinsics test separate from the pure java test.
ferakocz Mar 12, 2025
2438fb5
responding to review comments
ferakocz Mar 20, 2025
1cfab77
More beautification
ferakocz Mar 20, 2025
e9db09e
Fix windows build
ferakocz Mar 20, 2025
9597b53
Added asserts for array sizes
ferakocz Mar 21, 2025
5665689
Further readability improvements.
ferakocz Mar 22, 2025
7a9f664
Reacting to comments by Volodymyr.
ferakocz Mar 31, 2025
e4ab10b
Reacting to comment by Sandhya.
ferakocz Apr 2, 2025
0b0d096
Reacting to mor comments from Sandhya.
ferakocz Apr 8, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions test/jdk/sun/security/provider/acvp/Launcher.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,14 @@
* @library /test/lib
* @modules java.base/sun.security.provider
* @run main Launcher
*/

/*
* @test
* @summary Test verifying the intrinsic implementation.
* @bug 8342442 8345057
* @library /test/lib
* @modules java.base/sun.security.provider
* @run main/othervm -Xcomp Launcher
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for adding this case. Please add it as a separate testcase:
/*

  • @test
  • @summary Test verifies intrinsic implementation.
  • @library /test/lib
  • @modules java.base/sun.security.provider
  • @run main/othervm -Xcomp Launcher
    */

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

*/

Expand Down