Skip to content

[BSC] support for Moran fork#5696

Merged
AlexeyAkhunov merged 5 commits into
erigontech:develfrom
14MR:proofop_moran_hardfork
Oct 11, 2022
Merged

[BSC] support for Moran fork#5696
AlexeyAkhunov merged 5 commits into
erigontech:develfrom
14MR:proofop_moran_hardfork

Conversation

@14MR

@14MR 14MR commented Oct 11, 2022

Copy link
Copy Markdown
Contributor

This PR brings support for BSC Moran fork

@masayil

masayil commented Oct 11, 2022

Copy link
Copy Markdown

@14MR hi you forgot the bsc chapel testnet

@sjlee1125

Copy link
Copy Markdown

Does this commit cover chapel testnet?

@14MR

14MR commented Oct 11, 2022

Copy link
Copy Markdown
Contributor Author

@sjlee1125 @masayil updated the chainspec

@tsutsu

tsutsu commented Oct 11, 2022

Copy link
Copy Markdown
Contributor

@14MR you're missing this part of the patch:

diff --git a/core/vm/evm.go b/core/vm/evm.go
index c671e5c3e..33d0fb5ad 100644
--- a/core/vm/evm.go
+++ b/core/vm/evm.go
@@ -45,6 +45,8 @@ type (
 func (evm *EVM) precompile(addr common.Address) (PrecompiledContract, bool) {
        var precompiles map[common.Address]PrecompiledContract
        switch {
+       case evm.chainRules.IsMoran:
+               precompiles = PrecompiledContractsIsMoran
        case evm.chainRules.IsNano:
                precompiles = PrecompiledContractsNano
        case evm.chainRules.IsBerlin:

@14MR

14MR commented Oct 11, 2022

Copy link
Copy Markdown
Contributor Author

@14MR you're missing this part of the patch:

diff --git a/core/vm/evm.go b/core/vm/evm.go
index c671e5c3e..33d0fb5ad 100644
--- a/core/vm/evm.go
+++ b/core/vm/evm.go
@@ -45,6 +45,8 @@ type (
 func (evm *EVM) precompile(addr common.Address) (PrecompiledContract, bool) {
        var precompiles map[common.Address]PrecompiledContract
        switch {
+       case evm.chainRules.IsMoran:
+               precompiles = PrecompiledContractsIsMoran
        case evm.chainRules.IsNano:
                precompiles = PrecompiledContractsNano
        case evm.chainRules.IsBerlin:

Thank you, i also found a few missing parts in tests. Updated

@AlexeyAkhunov AlexeyAkhunov merged commit 70851c3 into erigontech:devel Oct 11, 2022
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.

5 participants