Add EIP: Precompile for NTT operations#9374
Merged
eth-bot merged 41 commits intoethereum:masterfrom Mar 17, 2026
Merged
Conversation
github-actions
bot
added
c-new
Collaborator
|
✅ All reviewers have approved. |
eth-bot
added
e-consensus
eth-bot
changed the title
github-actions
bot
added
w-ci
bomanaps
reviewed
Feb 18, 2025
bomanaps
left a comment
bomanaps
left a comment
There was a problem hiding this comment.
Please follow the recommendation provided by the bot to properly get rid of the error messages.
github-actions
bot
added
w-ci
SamWilsn
reviewed
Feb 18, 2025
rdubois-crypto
changed the title
- Replace Lattigo-based implementation with liboqs library implementation - Update gas costs: NTT (scheme-specific: 256-1080 gas), VECMULMOD (ceil(0.32×N)), VECADDMOD (ceil(0.3×N)) - Update benchmark results
3 tasks
g11tech
reviewed
Nov 11, 2025
g11tech
requested changes
Nov 11, 2025
Contributor
g11tech
left a comment
g11tech
left a comment
There was a problem hiding this comment.
- pls address the comments and the CI
- resolve the comments that you have addressed, also add the comment to them how you have resolved them
- the algorithm images in the assets could be replaced by the text code
Update EIP-7885: Migrate to liboqs implementation
- Reorganize op-geth assets into nocgo/ and cgo/ subdirectories - Update Gas Cost Analysis with comparative table for both variants - Update Reference Implementation section with both variants - Document Pure Go (nocgo) as default, liboqs (cgo) as high-performance option - Remove external links
g11tech
reviewed
Nov 20, 2025
Update EIP-7885: Add nocgo/cgo implementation variants
|
The commit 5d571d9 (as a parent of e49671c) contains errors. |
Contributor
|
pls resolve CI |
Update EIP-7885: Add end-to-end signature verification benchmarks for NTT precompiles
This was referenced Feb 10, 2026
g11tech
requested changes
Feb 17, 2026
Contributor
g11tech
left a comment
g11tech
left a comment
There was a problem hiding this comment.
please add the todo markers as suggested in the comments, as well as check the rendered markdown as it shows some math symbol rendering errors
- Add <!-- TODO --> markers to Constants and Security Considerations
- Change precompile addresses to TBD (assigned on acceptance)
- Remove extra blank lines between list items throughout
- Fix math rendering: \mathbb F_q → \mathbb{F}_q, use \cdot for multiplication, = → \equiv
- Fix typo: Denotoing → Denoting
Update EIP-7885: Address reviewer feedback
Update EIP-7885: Fix MD049 linter error for mathbb notation
g11tech
reviewed
Feb 28, 2026
EIPS/eip-7885.md
Outdated
| title: Precompile for NTT operations | ||
| description: Proposal to add a precompiled contract that performs number theoretical transformation (NTT) and inverse (InvNTT). | ||
| author: Renaud Dubois (@rdubois-crypto), Simon Masson (@simonmasson), Yoon Hyoung Lee (@yhl125) | ||
| discussions-to: https://ethereum-magicians.org/t/eip-9374-precompile-for-ntt-operations/22895 |
Contributor
There was a problem hiding this comment.
this discussion to url has incorrect eip number, pls fix the url and update here
Update EIP-7885: update discussion link
This was referenced Mar 10, 2026
g11tech
approved these changes
Mar 17, 2026
eth-bot
approved these changes
Mar 17, 2026
Collaborator
eth-bot
left a comment
eth-bot
left a comment
There was a problem hiding this comment.
All Reviewers Have Approved; Performing Automatic Merge...
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
8 participants
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.
This proposal creates a precompiled contract that performs NTT and Inverse NTT transformations. This provides a way to have efficient and fast polynomial multiplication for Post Quantum signatures and Starks settlement.