-
Notifications
You must be signed in to change notification settings - Fork 26
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: ModularIsEqualChip adapter + core + positive tests #743
Conversation
INT-2477 ISEQMOD_RV32<N> Chip
See #664 We were previously relying on the U256 SLTU and submod. This was not good because it had both some special assumptions about the prime but also will not work when |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
0158b19
to
46a5f08
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
0e9edce
to
292137c
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
f59c30d
to
6bb0ebb
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
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.
Nice solution for doing a combined less than! LGTM, after the comments
Detailed Metrics
Commit: 8f62c67 |
Detailed Metrics
Commit: 8f62c67 |
Resolves INT-2477