Skip to content

Add ML-DSA x86_64 native assembly to CC builder scripts#1110

Merged
jakemas merged 1 commit into
mainfrom
mldsa-native-x86-cc-builder
Apr 29, 2026
Merged

Add ML-DSA x86_64 native assembly to CC builder scripts#1110
jakemas merged 1 commit into
mainfrom
mldsa-native-x86-cc-builder

Conversation

@jakemas
Copy link
Copy Markdown
Contributor

@jakemas jakemas commented Apr 28, 2026

Summary

  • Add discovery of ML-DSA mldsa-native x86_64 assembly files in the CC builder scripts for linux_x86_64.sh and apple_x86_64.sh
  • Follows the same pattern as ML-KEM's native assembly discovery

Context

aws/aws-lc#3195 imports the mldsa-native x86_64 AVX2 backend into aws-lc, adding .S files under crypto/fipsmodule/ml_dsa/mldsa/native/x86_64/src/. The aws-lc-rs CC builder needs to discover these files, otherwise the aws-lc-rs-linux CI job in aws-lc fails. This PR unblocks aws/aws-lc#3195.

This is the same pattern used when ML-KEM's x86_64 backend was first added.

Changes

  • linux_x86_64.sh: Add find for ML-DSA x86_64 assembly
  • apple_x86_64.sh: Add find for ML-DSA x86_64 assembly

Note

The generated .rs builder files will need regeneration after the aws-lc submodule is bumped to include the ML-DSA native backend.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and the ISC license.

Add discovery of ML-DSA mldsa-native x86_64 assembly files in the CC
builder scripts for linux-x86_64 and apple-x86_64, matching the
existing pattern used for ML-KEM.

This is needed once aws-lc imports the mldsa-native x86_64 backend
(aws/aws-lc#3195).

Signed-off-by: Jake Massimo <jakemas@amazon.com>
@jakemas jakemas requested a review from a team as a code owner April 28, 2026 21:02
Copy link
Copy Markdown

@WillChilds-Klein WillChilds-Klein left a comment

Choose a reason for hiding this comment

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

do we not need similar updates to the windows x86_64 script?

@jakemas
Copy link
Copy Markdown
Contributor Author

jakemas commented Apr 28, 2026

No — the Windows script (win_x86_64.sh) only discovers .asm files from generated-src/win-x86_64/. The mldsa-native assembly files are .S (GAS syntax), not .asm (NASM/MASM). ML-KEM doesn't add its native .S files to the Windows script either — only the four Linux/Apple scripts discover native assembly. So this PR matches the existing pattern.

@jakemas jakemas merged commit 2e20f7c into main Apr 29, 2026
1261 of 1267 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants