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

Add GFNI tables #343

Closed
wants to merge 1 commit into from
Closed

Conversation

klauspost
Copy link
Contributor

GFNI instructions added as 'extras'.

See #335

I couldn't make the scripts work. Hopefully it is just a matter of re-generating.

GFNI instructions added as 'extras'.

See mmcloughlin#335
@mmcloughlin
Copy link
Owner

Thanks for this. I wasn't sure how to modify this PR (might need permission from you).

So anyway, I pulled it down and reran code generation. The result is pushed up as #344.

mmcloughlin added a commit that referenced this pull request Nov 28, 2022
Adds support for the GFNI "Galois Field New Instructions" instruction set.

These instructions are not included in the Opcodes database, therefore they're
added using the "extras" mechanism introduced in #345.

For simplicity, the loading phase is updated slightly so that AVX-512 form
expansion rules are applied after extras are added to the list. This greatly
reduces the number of forms that have to be specified by hand.

Based on #343
Fixes #335

Co-authored-by: Klaus Post <klauspost@gmail.com>
@mmcloughlin
Copy link
Owner

Landed in #344. I added you as a co-author in the commit. Thanks for the contribution!

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.

2 participants