optimize opcode:

- Optimize ``as_opcode()``
- Remove unnecessary copy methods from OpcodeAPI
- fix lint
- run / trigger CI

Author: charles-cooper

eth/_utils/numeric.py

@@ -1,5 +1,4 @@
 import decimal
-import functools
 import itertools
 from typing import (
     Union,
@@ -31,10 +30,11 @@ def int_to_bytes32(value: Union[int, bool]) -> Hash32:
 
 
 # hotspot, optimized
-def ceil32(x):
+def ceil32(x: int) -> int:
     return (x + 31) & ~31
 
-def ceil8(x):
+
+def ceil8(x: int) -> int:
     return (x + 7) & ~7
 
 

eth/abc.py

@@ -1507,26 +1507,12 @@ def as_opcode(
         logic_fn: Callable[["ComputationAPI"], None],
         mnemonic: str,
         gas_cost: int,
-    ) -> T:
+    ) -> "OpcodeAPI":
         """
         Class factory method for turning vanilla functions into Opcodes.
         """
         ...
 
-    @abstractmethod
-    def __copy__(self) -> "OpcodeAPI":
-        """
-        Return a copy of the opcode.
-        """
-        ...
-
-    @abstractmethod
-    def __deepcopy__(self, memo: Any) -> "OpcodeAPI":
-        """
-        Return a deep copy of the opcode.
-        """
-        ...
-
 
 class ChainContextAPI(ABC):
     """

eth/vm/forks/berlin/opcodes.py

@@ -10,6 +10,9 @@
 from eth import (
     constants,
 )
+from eth.abc import (
+    OpcodeAPI,
+)
 from eth.vm import (
     mnemonics,
     opcode_values,
@@ -24,15 +27,14 @@
     GAS_SELFDESTRUCT_EIP150,
 )
 from eth.vm.opcode import (
-    Opcode,
     as_opcode,
 )
 
 from . import (
     logic,
 )
 
-UPDATED_OPCODES: Dict[int, Opcode] = {
+UPDATED_OPCODES: Dict[int, OpcodeAPI] = {
     opcode_values.BALANCE: as_opcode(
         gas_cost=constants.GAS_NULL,
         logic_fn=logic.balance_eip2929,

eth/vm/forks/london/opcodes.py

@@ -10,6 +10,9 @@
 from eth import (
     constants,
 )
+from eth.abc import (
+    OpcodeAPI,
+)
 from eth.vm import (
     mnemonics,
     opcode_values,
@@ -24,15 +27,14 @@
     block,
 )
 from eth.vm.opcode import (
-    Opcode,
     as_opcode,
 )
 
 from . import (
     storage,
 )
 
-UPDATED_OPCODES: Dict[int, Opcode] = {
+UPDATED_OPCODES: Dict[int, OpcodeAPI] = {
     opcode_values.BASEFEE: as_opcode(
         gas_cost=constants.GAS_BASE,
         logic_fn=block.basefee,

eth/vm/logic/duplication.py

@@ -1,45 +1,67 @@
-import functools
-
 from eth.abc import (
     ComputationAPI,
 )
 
 
-def dup_XX(computation: ComputationAPI, position: int) -> None:
-    """
-    Stack item duplication.
-    """
-    computation.stack_dup(position)
-
-def dup1(computation):
-    dup_XX(computation, 1)
-def dup2(computation):
-    dup_XX(computation, 2)
-def dup3(computation):
-    dup_XX(computation, 3)
-def dup4(computation):
-    dup_XX(computation, 4)
-def dup5(computation):
-    dup_XX(computation, 5)
-def dup6(computation):
-    dup_XX(computation, 6)
-def dup7(computation):
-    dup_XX(computation, 7)
-def dup8(computation):
-    dup_XX(computation, 8)
-def dup9(computation):
-    dup_XX(computation, 9)
-def dup10(computation):
-    dup_XX(computation, 10)
-def dup11(computation):
-    dup_XX(computation, 11)
-def dup12(computation):
-    dup_XX(computation, 12)
-def dup13(computation):
-    dup_XX(computation, 13)
-def dup14(computation):
-    dup_XX(computation, 14)
-def dup15(computation):
-    dup_XX(computation, 15)
-def dup16(computation):
-    dup_XX(computation, 16)
+def dup1(computation: ComputationAPI) -> None:
+    computation.stack_dup(1)
+
+
+def dup2(computation: ComputationAPI) -> None:
+    computation.stack_dup(2)
+
+
+def dup3(computation: ComputationAPI) -> None:
+    computation.stack_dup(3)
+
+
+def dup4(computation: ComputationAPI) -> None:
+    computation.stack_dup(4)
+
+
+def dup5(computation: ComputationAPI) -> None:
+    computation.stack_dup(5)
+
+
+def dup6(computation: ComputationAPI) -> None:
+    computation.stack_dup(6)
+
+
+def dup7(computation: ComputationAPI) -> None:
+    computation.stack_dup(7)
+
+
+def dup8(computation: ComputationAPI) -> None:
+    computation.stack_dup(8)
+
+
+def dup9(computation: ComputationAPI) -> None:
+    computation.stack_dup(9)
+
+
+def dup10(computation: ComputationAPI) -> None:
+    computation.stack_dup(10)
+
+
+def dup11(computation: ComputationAPI) -> None:
+    computation.stack_dup(11)
+
+
+def dup12(computation: ComputationAPI) -> None:
+    computation.stack_dup(12)
+
+
+def dup13(computation: ComputationAPI) -> None:
+    computation.stack_dup(13)
+
+
+def dup14(computation: ComputationAPI) -> None:
+    computation.stack_dup(14)
+
+
+def dup15(computation: ComputationAPI) -> None:
+    computation.stack_dup(15)
+
+
+def dup16(computation: ComputationAPI) -> None:
+    computation.stack_dup(16)

eth/vm/logic/stack.py

@@ -1,5 +1,3 @@
-import functools
-
 from eth.abc import (
     ComputationAPI,
 )
@@ -24,67 +22,133 @@ def push_XX(computation: ComputationAPI, size: int) -> None:
         computation.stack_push_bytes(padded_value)
 
 
-def push1(computation):
+def push0(computation: ComputationAPI) -> None:
+    return push_XX(computation, 0)
+
+
+def push1(computation: ComputationAPI) -> None:
     return push_XX(computation, 1)
-def push2(computation):
+
+
+def push2(computation: ComputationAPI) -> None:
     return push_XX(computation, 2)
-def push3(computation):
+
+
+def push3(computation: ComputationAPI) -> None:
     return push_XX(computation, 3)
-def push4(computation):
+
+
+def push4(computation: ComputationAPI) -> None:
     return push_XX(computation, 4)
-def push5(computation):
+
+
+def push5(computation: ComputationAPI) -> None:
     return push_XX(computation, 5)
-def push6(computation):
+
+
+def push6(computation: ComputationAPI) -> None:
     return push_XX(computation, 6)
-def push7(computation):
+
+
+def push7(computation: ComputationAPI) -> None:
     return push_XX(computation, 7)
-def push8(computation):
+
+
+def push8(computation: ComputationAPI) -> None:
     return push_XX(computation, 8)
-def push9(computation):
+
+
+def push9(computation: ComputationAPI) -> None:
     return push_XX(computation, 9)
-def push10(computation):
+
+
+def push10(computation: ComputationAPI) -> None:
     return push_XX(computation, 10)
-def push11(computation):
+
+
+def push11(computation: ComputationAPI) -> None:
     return push_XX(computation, 11)
-def push12(computation):
+
+
+def push12(computation: ComputationAPI) -> None:
     return push_XX(computation, 12)
-def push13(computation):
+
+
+def push13(computation: ComputationAPI) -> None:
     return push_XX(computation, 13)
-def push14(computation):
+
+
+def push14(computation: ComputationAPI) -> None:
     return push_XX(computation, 14)
-def push15(computation):
+
+
+def push15(computation: ComputationAPI) -> None:
     return push_XX(computation, 15)
-def push16(computation):
+
+
+def push16(computation: ComputationAPI) -> None:
     return push_XX(computation, 16)
-def push17(computation):
+
+
+def push17(computation: ComputationAPI) -> None:
     return push_XX(computation, 17)
-def push18(computation):
+
+
+def push18(computation: ComputationAPI) -> None:
     return push_XX(computation, 18)
-def push19(computation):
+
+
+def push19(computation: ComputationAPI) -> None:
     return push_XX(computation, 19)
-def push20(computation):
+
+
+def push20(computation: ComputationAPI) -> None:
     return push_XX(computation, 20)
-def push21(computation):
+
+
+def push21(computation: ComputationAPI) -> None:
     return push_XX(computation, 21)
-def push22(computation):
+
+
+def push22(computation: ComputationAPI) -> None:
     return push_XX(computation, 22)
-def push23(computation):
+
+
+def push23(computation: ComputationAPI) -> None:
     return push_XX(computation, 23)
-def push24(computation):
+
+
+def push24(computation: ComputationAPI) -> None:
     return push_XX(computation, 24)
-def push25(computation):
+
+
+def push25(computation: ComputationAPI) -> None:
     return push_XX(computation, 25)
-def push26(computation):
+
+
+def push26(computation: ComputationAPI) -> None:
     return push_XX(computation, 26)
-def push27(computation):
+
+
+def push27(computation: ComputationAPI) -> None:
     return push_XX(computation, 27)
-def push28(computation):
+
+
+def push28(computation: ComputationAPI) -> None:
     return push_XX(computation, 28)
-def push29(computation):
+
+
+def push29(computation: ComputationAPI) -> None:
     return push_XX(computation, 29)
-def push30(computation):
+
+
+def push30(computation: ComputationAPI) -> None:
     return push_XX(computation, 30)
-def push31(computation):
+
+
+def push31(computation: ComputationAPI) -> None:
     return push_XX(computation, 31)
-def push32(computation):
+
+
+def push32(computation: ComputationAPI) -> None:
     return push_XX(computation, 32)