Skip to content

performance improvements #37

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.

Sign up for GitHub

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

Jump to bottom
Merged
adecaro merged 15 commits into from
Jan 23, 2026
Merged

performance improvements #37

adecaro merged 15 commits into from
Jan 23, 2026

Conversation

@adecaro
Copy link
Member

@adecaro adecaro commented Dec 15, 2025 •
edited
Loading

This PR introduces computation and memory allocation improvements for the curve gurvy/bls12381.
In addition, it adds the following functions to math.go:

  • Zr#InvModOrder to compute the inverse module the order of the curve this Zr element belongs to
  • G1#Mul2InPlace is like G1#Mul2 but mutative
  • ModAddMul sums the component-wise produce of the input vectors.
  • ModAddMul2 sums the products of first and second inputs and third and forth inputs

adecaro added 10 commits December 8, 2025 13:28
@adecaro
better Mul2
0cc4f07 
Signed-off-by: Angelo De Caro <adc@zurich.ibm.com>
@adecaro
dep update
1dd505f 
Signed-off-by: Angelo De Caro <adc@zurich.ibm.com>
@adecaro
better mul2
da56f33 
Signed-off-by: Angelo De Caro <adc@zurich.ibm.com>
@adecaro
improvements
bdbf97a 
Signed-off-by: Angelo De Caro <adc@zurich.ibm.com>
@adecaro
better performance and memory allocation for bls12381
7e58831 
Signed-off-by: Angelo De Caro <adc@zurich.ibm.com>
@adecaro
InvModOrder
86dd03c 
Signed-off-by: Angelo De Caro <adc@zurich.ibm.com>
@adecaro
improvements
6050704 
Signed-off-by: Angelo De Caro <adc@zurich.ibm.com>
@adecaro
fix PowMod
4c443f1 
Signed-off-by: Angelo De Caro <adc@zurich.ibm.com>
@adecaro
mod operations fix
f4de1d2 
Signed-off-by: Angelo De Caro <adc@zurich.ibm.com>
@adecaro
unit-tests + fix
08690b7 
Signed-off-by: Angelo De Caro <adc@zurich.ibm.com>
@adecaro adecaro self-assigned this Dec 15, 2025
@adecaro adecaro requested a review from ale-linux December 15, 2025 09:49
ale-linux
ale-linux reviewed Dec 15, 2025
View reviewed changes
Copy link
Collaborator

@ale-linux ale-linux left a comment

Choose a reason for hiding this comment

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

LGTM - only one Q

@adecaro
cleanup + dep update
bd86b4e 
Signed-off-by: Angelo De Caro <adc@zurich.ibm.com>
@adecaro adecaro requested a review from ale-linux January 14, 2026 09:10
@adecaro
reset v before putting it back
062c164 
Signed-off-by: Angelo De Caro <adc@zurich.ibm.com>
@adecaro
make the pool private
8e24692 
Signed-off-by: Angelo De Caro <adc@zurich.ibm.com>
@adecaro
add reset to g1 pool too
2bc955c 
Signed-off-by: Angelo De Caro <adc@zurich.ibm.com>
@adecaro
test for InvModOrder
f52c9be 
Signed-off-by: Angelo De Caro <adc@zurich.ibm.com>
@adecaro adecaro merged commit 21c357e into main Jan 23, 2026
2 checks passed
@adecaro adecaro deleted the f-perf branch January 23, 2026 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ale-linux ale-linux ale-linux approved these changes

Assignees

@adecaro adecaro

Labels

performance

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@adecaro @ale-linux