-
Notifications
You must be signed in to change notification settings - Fork 1
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
impl multiplication for polyval #55
Merged
Merged
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
3698750
impl bmul multiplication subroutine
thor314 0c4fd52
impl gfmul (mostly)
thor314 b706432
implement rest of mul
thor314 e9e4495
notes on rev64 added, renamed mul64 -> wrappingMul64 for clarity
thor314 b839880
wrapping mul partial impl 1 - testing in progress
thor314 02a1830
wrapping mul tests correctly running, not passing
thor314 8980c89
rm component
thor314 bed8b7a
debug wrapping mul - pause - tests for wrapping mul failing
thor314 09a7674
impl LE wrapping mul, tests pass
thor314 ec817c8
parse int le/be
thor314 35549ee
Switched suite name from Wrapping_LE to Wrapping_BE
KaiGeffen c6e57c5
Merge https://github.com/pluto/aes-proof into impl-polymul
KaiGeffen 8a18505
wrapping mul moved
thor314 0176062
Merge branch 'impl-polymul' of https://github.com/pluto/aes-proof int…
KaiGeffen dcc42ae
Removed unused test impl / resolved test todos
KaiGeffen 0fb4b23
Added wrap_mul test, fixed minor scope smell
KaiGeffen 177658b
tests written for bmul
thor314 f8fe80b
bugfix: indexing error in BMUL
thor314 42fe433
tests for gfmul
thor314 3046074
annotate gfmul empty bits issue
thor314 a37608c
gfmul debugged
thor314 c585377
adjusted expected values in gfmul tests; circomkit still only capturi…
thor314 b11e9be
gfmul tests fixed
thor314 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -86,4 +86,4 @@ template Mul() | |
} | ||
} | ||
|
||
} | ||
} |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Is this file needed?