Skip to content

Conversation

marcindsobczak
Copy link
Contributor

@marcindsobczak marcindsobczak commented Jul 2, 2025

Tripling the price for ModExp calls.

Moving worst-performing ModExp test case from 18 to about 31MGas/s.

@github-actions github-actions bot added c-update Modifies an existing proposal s-draft This EIP is a Draft t-core labels Jul 2, 2025
@eth-bot
Copy link
Collaborator

eth-bot commented Jul 2, 2025

✅ All reviewers have approved.

@marcindsobczak marcindsobczak changed the title Triple price.md Update EIP-7883: Triple price.md Jul 2, 2025
@marcindsobczak marcindsobczak changed the title Update EIP-7883: Triple price.md Update EIP-7883: Triple price Jul 2, 2025
| modexp_nagydani_4_pow0x10001 | 21845 | 131072 | 500% |
| modexp_nagydani_5_square | 5461 | 32768 | 500% |
| modexp_nagydani_5_qube | 5461 | 32768 | 500% |
| modexp_nagydani_5_pow0x10001 | 87381 | 525288 | 500% |
Copy link
Contributor

Choose a reason for hiding this comment

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

We got 524288 for modexp_nagydani_5_pow0x10001. It's the only discrepancy we have. Should this be the case? I am double checking now.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have 524288 too, so it was a typo. I will fix it in a moment

@eth-bot eth-bot enabled auto-merge (squash) July 11, 2025 10:56
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 271c8a7 into ethereum:master Jul 11, 2025
16 checks passed
chfast added a commit to ipsilon/evmone that referenced this pull request Jul 19, 2025
Update the [EIP-7883 "ModExp Gas Cost Increase"](https://eips.ethereum.org/EIPS/eip-7883)
to the latest revision:
- ethereum/EIPs#9855
- ethereum/EIPs#9969
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c-update Modifies an existing proposal s-draft This EIP is a Draft t-core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants