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.

Sign up for GitHub

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

Jump to bottom

perf: minor optimization of byte opcode function #156

Open

tommyrharper wants to merge 2 commits into rust-ethereum:master from tommyrharper:optimize-byte-opcode

tommyrharper commented Mar 3, 2023 •

edited

Loading

I have added a minor optimization to the BYTE evaluator.

Essentially we can skip the for loop entirely if op1 < 32, and we only need to iterate up to 7, not 255.

So this brings it down from 256 iterations for each function call to either 0 or 8 iterations depending on the value of op1.

tommyrharper added 2 commits

March 3, 2023 10:42


          perf: minor optimization of byte opcode function

37bff8b


          refactor: remove byte early return statement

68ecfd3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet