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.

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

Rework main loop iteration #107

Merged
merged 2 commits into from
Jul 24, 2019
Merged

Rework main loop iteration #107

merged 2 commits into from
Jul 24, 2019

Conversation

chfast
Copy link
Member

@chfast chfast commented Jul 24, 2019 •
edited
Loading 

Comparing bin/evmone-bench-base to bin/evmone-bench
Benchmark                            Time             CPU      Time Old      Time New       CPU Old       CPU New
-----------------------------------------------------------------------------------------------------------------
sha1_shifts/empty                 -0.0650         -0.0650            49            46            49            46
sha1_shifts/1351                  -0.0841         -0.0841           933           855           933           855
sha1_shifts/2737                  -0.0843         -0.0843          1817          1664          1817          1664
sha1_shifts/5311                  -0.0838         -0.0838          3541          3244          3541          3244
sha1_shifts/65536                 -0.0837         -0.0838         43144         39532         43144         39529
blake2b_huff/empty                -0.0278         -0.0277            63            61            63            61
blake2b_huff/abc                  -0.0314         -0.0314            63            61            63            61
blake2b_huff/2805nulls            -0.0787         -0.0788           462           426           462           426
blake2b_huff/2805aa               -0.0795         -0.0795           462           426           462           426
blake2b_huff/5610nulls            -0.0817         -0.0817           862           791           862           791
blake2b_huff/8415nulls            -0.0853         -0.0853          1242          1136          1242          1136
blake2b_huff/65536nulls           -0.0874         -0.0874          9362          8544          9362          8544
sha1_divs/empty                   -0.0390         -0.0390            96            93            96            93
sha1_divs/1351                    -0.0438         -0.0438          1908          1824          1908          1824
sha1_divs/2737                    -0.0430         -0.0430          3715          3555          3715          3555
sha1_divs/5311                    -0.0427         -0.0427          7247          6937          7247          6937
sha1_divs/65536                   -0.0391         -0.0391         88134         84690         88134         84689
blake2b_shifts/2805nulls          -0.0429         -0.0429          6950          6652          6950          6652
blake2b_shifts/5610nulls          -0.0370         -0.0370         13961         13445         13962         13445
blake2b_shifts/8415nulls          -0.0427         -0.0427         20848         19958         20848         19957
blake2b_shifts/65536nulls         -0.0894         -0.0894        163352        148756        163354        148757

@codecov-io
Copy link

codecov-io commented Jul 24, 2019 •
edited
Loading

Codecov Report

Merging #107 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #107   +/-   ##
=======================================
  Coverage   88.57%   88.57%           
=======================================
  Files          18       18           
  Lines        1865     1865           
  Branches      197      196    -1     
=======================================
  Hits         1652     1652           
  Misses        188      188           
  Partials       25       25

@chfast chfast requested review from axic and gumb0 July 24, 2019 15:40
gumb0
gumb0 approved these changes Jul 24, 2019
View reviewed changes
Copy link
Member

@gumb0 gumb0 left a comment

Choose a reason for hiding this comment

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

Nice, it got simpler, too

@chfast chfast merged commit d83bdfb into master Jul 24, 2019
@chfast chfast deleted the main_loop branch July 24, 2019 21:12
@chfast chfast restored the main_loop branch July 24, 2019 21:14
@axic axic deleted the main_loop branch July 25, 2019 11:06
jwasinger pushed a commit to jwasinger/evmone that referenced this pull request Apr 27, 2021
@chfast
Merge pull request ethereum#107 from ethereum/docs-vmtester
a503fcf 
docs: Add basic information about evmc-vmtester
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gumb0 gumb0 gumb0 approved these changes

@axic axic Awaiting requested review from axic

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@chfast @codecov-io @gumb0