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

Simplify commitee weight computation #3246

Merged
merged 1 commit into from
Feb 13, 2023

Conversation

potuz
Copy link
Contributor

@potuz potuz commented Feb 10, 2023

Simplify unnecessary divisions since there is no possible overflow

Copy link
Contributor

@djrtwo djrtwo left a comment

Choose a reason for hiding this comment

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

This is the original PR https://github.com/ethereum/consensus-specs/pull/2730/files

No discussion of why the indirection and intermediate vars... I think was just an oversight

Approved

@djrtwo djrtwo merged commit 5970ae5 into ethereum:dev Feb 13, 2023
etan-status added a commit to status-im/nimbus-eth2 that referenced this pull request May 5, 2023
Optimizations from ethereum/consensus-specs#3246
avoid a couple unnecessary divisions when calculating proposer boost.
tersec pushed a commit to status-im/nimbus-eth2 that referenced this pull request May 9, 2023
Optimizations from ethereum/consensus-specs#3246
avoid a couple unnecessary divisions when calculating proposer boost.
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.

2 participants