Skip to content
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.

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

Implement LLVM x86 gfni intrinsics #3895

Merged
merged 1 commit into from
Oct 4, 2024
Merged

Implement LLVM x86 gfni intrinsics #3895

merged 1 commit into from
Oct 4, 2024

Conversation

TDecking
Copy link
Contributor

No description provided.

@TDecking
Copy link
Contributor Author

@rustbot ready

@rustbot rustbot added the S-waiting-on-review Status: Waiting for a review to complete label Sep 17, 2024
Copy link
Member

@RalfJung RalfJung left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! Are these the (in)famous NSA instructions in the x86 instruction set? They surely look sufficiently obscure. ;)

Unfortunately, what these instructions do is complete gibberish to me. Like, I know the basics of finite ring/field arithmetic, but I stare at the test file and nothing makes any sense.^^ This requires already knowing what all these instructions do to make any sense of -- which I don't, and I won't have the time to learn these either. Is there any way to make this test file a bit more approachable to someone who knows Rust and rings "modulo N" but nothing else? By adding comments that map all the Intel idiosyncrasies to something that makes mathematical sense, or so?

src/shims/x86/gfni.rs Outdated Show resolved Hide resolved
src/shims/x86/gfni.rs Outdated Show resolved Hide resolved
src/shims/x86/gfni.rs Outdated Show resolved Hide resolved
src/shims/x86/gfni.rs Outdated Show resolved Hide resolved
tests/pass/shims/x86/intrinsics-x86-gfni.rs Show resolved Hide resolved
tests/pass/shims/x86/intrinsics-x86-gfni.rs Show resolved Hide resolved
src/shims/x86/gfni.rs Outdated Show resolved Hide resolved
src/shims/x86/gfni.rs Outdated Show resolved Hide resolved
@RalfJung
Copy link
Member

RalfJung commented Oct 4, 2024

Looks good, as far as I can tell... I still don't understand most of this code, but worst-case we get some complains that the intrinsics are doing the wrong thing. ;)

Thanks for the PR!
@bors r+

@bors
Copy link
Contributor

bors commented Oct 4, 2024

📌 Commit 71567cc has been approved by RalfJung

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Oct 4, 2024

⌛ Testing commit 71567cc with merge 5801cef...

@bors
Copy link
Contributor

bors commented Oct 4, 2024

☀️ Test successful - checks-actions
Approved by: RalfJung
Pushing 5801cef to master...

@bors bors merged commit 5801cef into rust-lang:master Oct 4, 2024
8 checks passed
@TDecking TDecking deleted the gfni branch October 4, 2024 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Waiting for a review to complete
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants