-
Notifications
You must be signed in to change notification settings - Fork 224
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
feat: add packed mersenne31 #396
Conversation
2cd68e4
to
fa8c9b6
Compare
e67f22f
to
11c2b6b
Compare
c4b4c7b
to
11c2b6b
Compare
Aditionally, the function now accepts not only vector types but also any container type for increased flexibility.
11c2b6b
to
dbdbf3b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please fix the typo in de14c3b commit body:
Aditionally -> Additionally
tachyon/math/finite_fields/mersenne31/packed_mersenne31_avx512.h
Outdated
Show resolved
Hide resolved
b91dccc
to
13338bf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
See https://github.com/Plonky3/Plonky3/blob/6034010ae178439fffee55fd71f8105f62577a35/mersenne-31/src/x86_64_avx2/packing.rs, https://github.com/Plonky3/Plonky3/blob/6034010ae178439fffee55fd71f8105f62577a35/mersenne-31/src/x86_64_avx512/packing.rs and https://github.com/Plonky3/Plonky3/blob/6034010ae178439fffee55fd71f8105f62577a35/mersenne-31/src/aarch64_neon/packing.rs.
Since #387, `Mersenne31` is instantiated with `small_prime_field_generic.h` and all the mathematical methods become constexpr.
13338bf
to
47e6217
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
This PR adds PackedPrimeField for
Mersenne31
.