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

Update EIP-2537: Rename multiexponentiation to MSM #8393

Merged
merged 2 commits into from
Apr 6, 2024

Conversation

mratsim
Copy link
Contributor

@mratsim mratsim commented Apr 6, 2024

Discussion: https://ethereum-magicians.org/t/eip-2537-bls12-precompile-discussion-thread/4187/63

The spec is inconsistent and uses both additive notation (G!ADD and G1MUL) and multiplicative notation (G!MULTIEXP) for the elliptic curve group law.

Blockchain, cryptography and ZK research has settled in using the additive notation.
And previous BN254 EIPs (EIP196) already uses the additive notation.

@mratsim mratsim requested a review from eth-bot as a code owner April 6, 2024 13:20
@github-actions github-actions bot added c-update Modifies an existing proposal s-review This EIP is in Review t-core labels Apr 6, 2024
@eth-bot
Copy link
Collaborator

eth-bot commented Apr 6, 2024

✅ All reviewers have approved.

@eth-bot eth-bot added the a-review Waiting on author to review label Apr 6, 2024
@eth-bot eth-bot changed the title Update eip-2537: Rename multiexponentiation to MSM Update EIP-2537: Rename multiexponentiation to MSM Apr 6, 2024
Copy link
Member

@ralexstokes ralexstokes left a comment

Choose a reason for hiding this comment

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

looks good, open to this change

EIPS/eip-2537.md Outdated Show resolved Hide resolved
@asanso
Copy link
Contributor

asanso commented Apr 6, 2024

+1 from my side

Co-authored-by: Alex Stokes <r.alex.stokes@gmail.com>
Copy link
Member

@ralexstokes ralexstokes left a comment

Choose a reason for hiding this comment

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

looks good ,thank you!

@eth-bot eth-bot enabled auto-merge (squash) April 6, 2024 15:25
Copy link
Collaborator

@eth-bot eth-bot left a comment

Choose a reason for hiding this comment

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

All Reviewers Have Approved; Performing Automatic Merge...

@eth-bot eth-bot merged commit b6b2b84 into ethereum:master Apr 6, 2024
10 checks passed
Copy link

@Gemrav Gemrav left a comment

Choose a reason for hiding this comment

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

H

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a-review Waiting on author to review c-update Modifies an existing proposal s-review This EIP is in Review t-core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants