Skip to content
This repository was archived by the owner on Apr 8, 2026. It is now read-only.

instructions: Set gas cost for undefined instructions to 0#425

Merged
chfast merged 2 commits intomasterfrom
instruction_metrics
Sep 19, 2019
Merged

instructions: Set gas cost for undefined instructions to 0#425
chfast merged 2 commits intomasterfrom
instruction_metrics

Conversation

@chfast
Copy link
Copy Markdown
Member

@chfast chfast commented Sep 19, 2019

Closes #356.

@chfast chfast force-pushed the instruction_metrics branch from b78eb52 to e87b24d Compare September 19, 2019 14:07
/* REVERT = 0xfd */ {ZERO, 2, 0},
/* INVALID = 0xfe */ {ZERO, 0, 0},
/* SUICIDE = 0xff */ {5000, 1, 0},
/* SELFDESTRUCT = 0xff */ {5000, 1, 0},
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.

Wow this wasn't done yet? :)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Only in comments. This was created by transforming Aleth's table / enum.

@chfast chfast merged commit b831a16 into master Sep 19, 2019
@chfast chfast deleted the instruction_metrics branch September 19, 2019 14:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Instructions: consider definding undefined instruction's cost as 0 instead of -1

2 participants