Skip to content

Upgrade EVMC to version pre 7.0.0#191

Merged
chfast merged 4 commits intomasterfrom
evmc_instruction_metrics
Oct 31, 2019
Merged

Upgrade EVMC to version pre 7.0.0#191
chfast merged 4 commits intomasterfrom
evmc_instruction_metrics

Conversation

@chfast
Copy link
Copy Markdown
Member

@chfast chfast commented Sep 27, 2019

No description provided.

@codecov-io
Copy link
Copy Markdown

codecov-io commented Sep 27, 2019

Codecov Report

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

@@           Coverage Diff           @@
##           master     #191   +/-   ##
=======================================
  Coverage   99.88%   99.88%           
=======================================
  Files           5        5           
  Lines         839      839           
  Branches      109      109           
=======================================
  Hits          838      838           
  Misses          1        1

@chfast chfast force-pushed the evmc_instruction_metrics branch from 04e6c98 to e440844 Compare October 31, 2019 11:44
@chfast chfast changed the title Update comparison with EVMC's instruction metrics Upgrade EVMC to version pre 7.0.0 Oct 31, 2019
@chfast chfast marked this pull request as ready for review October 31, 2019 11:44
@chfast chfast requested review from axic and gumb0 October 31, 2019 11:44
@chfast chfast force-pushed the evmc_instruction_metrics branch from 041b17d to 86522b8 Compare October 31, 2019 11:49
table[OP_BALANCE] = {op_balance, 700, 1, 0};
table[OP_CHAINID] = {op_undefined, 2, 0, 1};
table[OP_EXTCODEHASH] = {op_extcodehash, 700, 1, 0};
table[OP_SELFBALANCE] = {op_undefined, 5, 0, 1};
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Shouldn't this and chainid be implemented?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

They are in separate PRs, it's better to merge this first, and then rebase those

@chfast chfast merged commit 6326761 into master Oct 31, 2019
@chfast chfast deleted the evmc_instruction_metrics branch October 31, 2019 14:05
@chfast chfast mentioned this pull request Oct 31, 2019
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.

4 participants