Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

removed redundant VMType enum with one variant #11266

Merged
merged 2 commits into from
Nov 20, 2019
Merged

removed redundant VMType enum with one variant #11266

merged 2 commits into from
Nov 20, 2019

Conversation

debris
Copy link
Collaborator

@debris debris commented Nov 15, 2019

In the past (3-4 years ago), parity supported 2 implementations of evm. rust interpreter and cpp-jit. Since jit is not supported for a very long time, VMType enum with one variant is completely redundant

@debris debris added A0-pleasereview 🤓 Pull request needs code review. M4-core ⛓ Core client code / Rust. labels Nov 15, 2019
@debris debris requested a review from dvdplm November 15, 2019 12:27
@ordian
Copy link
Collaborator

ordian commented Nov 15, 2019

It would be nice to be able to plug custom evm impls (like https://github.com/ethereum/evmone) via https://github.com/ethereum/evmc, wdyt?

Copy link
Collaborator

@dvdplm dvdplm left a comment

Choose a reason for hiding this comment

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

lgtm.

One question I have is if the evm::Factory is used at this point? I'm not sure what the evm_cache is used for though?

Copy link
Collaborator

@niklasad1 niklasad1 left a comment

Choose a reason for hiding this comment

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

LGTM

@debris
Copy link
Collaborator Author

debris commented Nov 19, 2019

One question I have is if the evm::Factory is used at this point? I'm not sure what the evm_cache is used for though?

I'm not sure. I'll look it

It would be nice to be able to plug custom evm impls (like https://github.com/ethereum/evmone) via https://github.com/ethereum/evmc, wdyt?

Of course, but this is just a nice to have feature. If anyone really wants it and commits to the compatibility maintenance of it, we can add it.

@debris debris merged commit e0091c6 into master Nov 20, 2019
@debris debris deleted the no-evm-type branch November 20, 2019 03:11
dvdplm added a commit that referenced this pull request Nov 20, 2019
* master:
  Clarify what first_block `None` means (#11269)
  removed redundant VMType enum with one variant (#11266)
  Ensure jsonrpc threading settings are sane (#11267)
  Return Ok(None) when the registrar contract returns empty slice (#11257)
  Add a benchmark for snapshot::account::to_fat_rlps() (#11185)
  Fix misc compile warnings (#11258)
  simplify verification (#11249)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-pleasereview 🤓 Pull request needs code review. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants