Skip to content

feat: add Isthmus BLS precompiles#472

Closed
meyer9 wants to merge 1 commit intoethereum-optimism:optimismfrom
meyer9:meyer9/add-isthmus-precompiles
Closed

feat: add Isthmus BLS precompiles#472
meyer9 wants to merge 1 commit intoethereum-optimism:optimismfrom
meyer9:meyer9/add-isthmus-precompiles

Conversation

@meyer9
Copy link
Copy Markdown
Contributor

@meyer9 meyer9 commented Jan 23, 2025

Description

Adds precompiles for BLS operations as specified in EIP-2537.

Depends on #471 (and has changes from this since I can't change the base of the PR)

@tynes
Copy link
Copy Markdown
Contributor

tynes commented Jan 24, 2025

You can rebase this now that #471 has been merged

@meyer9 meyer9 force-pushed the meyer9/add-isthmus-precompiles branch from b7041d3 to 1b2933e Compare January 24, 2025 17:50
Make sure g1 add implemented
@meyer9 meyer9 force-pushed the meyer9/add-isthmus-precompiles branch from 1b2933e to 12e8db8 Compare January 30, 2025 18:05
@tynes
Copy link
Copy Markdown
Contributor

tynes commented Feb 3, 2025

We still need to add in the input size restrictions

@refcell
Copy link
Copy Markdown
Contributor

refcell commented Feb 4, 2025

We still need to add in the input size restrictions

flagging @meyer9

@meyer9
Copy link
Copy Markdown
Contributor Author

meyer9 commented Feb 4, 2025

Added here - will rebase once this is merged: #490

@Inphi
Copy link
Copy Markdown
Contributor

Inphi commented Feb 5, 2025

@meyer9 https://github.com/ethereum-optimism/op-geth/pull/490/files seems to implement this. Is this PR still needed? I'm cutting op-geth releases so we can use these precompiles in the op-program and I'd like to know if there's any remaining code that needs to be merged?

@meyer9
Copy link
Copy Markdown
Contributor Author

meyer9 commented Feb 5, 2025

hmm yeah that one was supposed to be based on this one, so this is technically already merged!

@meyer9 meyer9 closed this Feb 5, 2025
@meyer9 meyer9 deleted the meyer9/add-isthmus-precompiles branch February 5, 2025 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants