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

Analysis optimization - use switch() #121

Merged
merged 5 commits into from
Aug 12, 2019
Merged

Analysis optimization - use switch() #121

merged 5 commits into from
Aug 12, 2019

Conversation

chfast
Copy link
Member

@chfast chfast commented Aug 9, 2019 •
edited
Loading 

Comparing bin/evmone-bench-master to bin/evmone-bench
Benchmark                          Time             CPU      Time Old      Time New       CPU Old       CPU New
---------------------------------------------------------------------------------------------------------------
sha1_shifts/analysis            -0.1639         -0.1638             4             3             4             3
stop/analysis                   -0.1656         -0.1656             0             0             0             0
blake2b_huff/analysis           -0.1537         -0.1536            41            35            41            35
sha1_divs/analysis              -0.1559         -0.1559             4             4             4             4
weierstrudel/analysis           -0.1504         -0.1504            49            41            49            41
blake2b_shifts/analysis         -0.1892         -0.1892            20            16            20            16

@codecov-io
Copy link

codecov-io commented Aug 9, 2019 •
edited
Loading

Codecov Report

Merging #121 into master will increase coverage by 4.32%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #121      +/-   ##
==========================================
+ Coverage   84.86%   89.18%   +4.32%     
==========================================
  Files          19       19              
  Lines        2022     2035      +13     
  Branches      221      218       -3     
==========================================
+ Hits         1716     1815      +99     
+ Misses        280      194      -86     
  Partials       26       26

@chfast chfast requested review from axic and gumb0 and removed request for axic August 9, 2019 15:03
@chfast chfast merged commit 77ebf0a into master Aug 12, 2019
@chfast chfast deleted the analysis-opt branch August 12, 2019 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gumb0 gumb0 Awaiting requested review from gumb0

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

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