monorepo: remove EIP 2315 (JUMPSUB)

packages/common/src/eips.ts

@@ -52,27 +52,6 @@ export const EIPs: EIPsDict = {
       },
     },
   },
-  2315: {
-    comment: 'Simple subroutines for the EVM',
-    url: 'https://eips.ethereum.org/EIPS/eip-2315',
-    status: Status.Draft,
-    minimumHardfork: Hardfork.Istanbul,
-    requiredEIPs: [],
-    gasPrices: {
-      beginsub: {
-        v: 2,
-        d: 'Base fee of the BEGINSUB opcode',
-      },
-      returnsub: {
-        v: 5,
-        d: 'Base fee of the RETURNSUB opcode',
-      },
-      jumpsub: {
-        v: 10,
-        d: 'Base fee of the JUMPSUB opcode',
-      },
-    },
-  },
   2565: {
     comment: 'ModExp gas cost',
     url: 'https://eips.ethereum.org/EIPS/eip-2565',

packages/common/test/eips.spec.ts

@@ -43,16 +43,6 @@ describe('[Common/EIPs]: Initialization / Chain params', () => {
     */
   })
 
-  it('isActivatedEIP()', () => {
-    let c = new Common({ chain: Chain.Mainnet, hardfork: Hardfork.Istanbul })
-    assert.equal(c.isActivatedEIP(2315), false, 'istanbul, eips: [] -> false (EIP-2315)')
-    c = new Common({ chain: Chain.Mainnet, hardfork: Hardfork.Istanbul, eips: [2315] })
-    assert.equal(c.isActivatedEIP(2315), true, 'istanbul, eips: [2315] -> true (EIP-2315)')
-    c = new Common({ chain: Chain.Mainnet, hardfork: Hardfork.Berlin })
-    assert.equal(c.isActivatedEIP(2929), true, 'berlin, eips: [] -> true (EIP-2929)')
-    assert.equal(c.isActivatedEIP(2315), false, 'berlin, eips: [] -> true (EIP-2315)')
-  })
-
   it('eipBlock', () => {
     const c = new Common({ chain: Chain.Mainnet })
 

packages/evm/src/evm.ts

@@ -192,8 +192,8 @@ export class EVM implements EVMInterface {
 
     // Supported EIPs
     const supportedEIPs = [
-      1153, 1559, 2315, 2565, 2718, 2929, 2930, 2935, 3074, 3198, 3529, 3540, 3541, 3607, 3651,
-      3670, 3855, 3860, 4399, 4895, 4788, 4844, 5133, 5656, 6780, 6800, 7516,
+      1153, 1559, 2565, 2718, 2929, 2930, 2935, 3074, 3198, 3529, 3540, 3541, 3607, 3651, 3670,
+      3855, 3860, 4399, 4895, 4788, 4844, 5133, 5656, 6780, 6800, 7516,
     ]
 
     for (const eip of this.common.eips()) {

packages/evm/src/interpreter.ts

@@ -79,7 +79,6 @@ export interface RunState {
   memoryWordCount: bigint
   highestMemCost: bigint
   stack: Stack
-  returnStack: Stack
   code: Uint8Array
   shouldDoJumpAnalysis: boolean
   validJumps: Uint8Array // array of values where validJumps[index] has value 0 (default), 1 (jumpdest), 2 (beginsub)
@@ -105,7 +104,6 @@ export interface InterpreterStep {
   gasRefund: bigint
   stateManager: EVMStateManagerInterface
   stack: bigint[]
-  returnStack: bigint[]
   pc: number
   depth: number
   opcode: {
@@ -164,7 +162,6 @@ export class Interpreter {
       memoryWordCount: BIGINT_0,
       highestMemCost: BIGINT_0,
       stack: new Stack(),
-      returnStack: new Stack(1023), // 1023 return stack height limit per EIP 2315 spec
       code: new Uint8Array(0),
       validJumps: Uint8Array.from([]),
       cachedPushes: {},
@@ -413,7 +410,6 @@ export class Interpreter {
         isAsync: opcode.isAsync,
       },
       stack: this._runState.stack.getStack(),
-      returnStack: this._runState.returnStack.getStack(),
       depth: this._env.depth,
       address: this._env.address,
       account: this._env.contract,
@@ -493,9 +489,6 @@ export class Interpreter {
         } else if (opcode === 0x5b) {
           // Define a JUMPDEST as a 1 in the valid jumps array
           jumps[i] = 1
-        } else if (opcode === 0x5c) {
-          // Define a BEGINSUB as a 2 in the valid jumps array
-          jumps[i] = 2
         }
       }
     }

packages/evm/src/opcodes/codes.ts

@@ -274,14 +274,6 @@ const eipOpcodes: { eip: number; opcodes: OpcodeEntry }[] = [
       0x5d: { name: 'TSTORE', isAsync: false, dynamicGas: false },
     },
   },
-  {
-    eip: 2315,
-    opcodes: {
-      0x5c: { name: 'BEGINSUB', isAsync: false, dynamicGas: false },
-      0x5d: { name: 'RETURNSUB', isAsync: false, dynamicGas: false },
-      0x5e: { name: 'JUMPSUB', isAsync: false, dynamicGas: false },
-    },
-  },
   {
     eip: 3198,
     opcodes: {

packages/evm/src/opcodes/functions.ts

@@ -43,7 +43,6 @@
   fromTwos,
   getDataSlice,
   jumpIsValid,
-  jumpSubIsValid,
   mod,
   toTwos,
   trap,
@@ -861,81 +860,46 @@
   ],
   // 0x5b: JUMPDEST
   [0x5b, function () {}],
-  // 0x5c: BEGINSUB (EIP 2315) / TLOAD (EIP 1153)
+  // 0x5c: TLOAD (EIP 1153)
   [
     0x5c,
-    function (runState, common) {
-      if (common.isActivatedEIP(2315)) {
-        // BEGINSUB
-        trap(ERROR.INVALID_BEGINSUB + ' at ' + describeLocation(runState))
-      } else if (common.isActivatedEIP(1153)) {
-        // TLOAD
-        const key = runState.stack.pop()
-        const keyBuf = setLengthLeft(bigIntToBytes(key), 32)
-        const value = runState.interpreter.transientStorageLoad(keyBuf)
-        const valueBN = value.length ? bytesToBigInt(value) : BIGINT_0
-        runState.stack.push(valueBN)
-      }
+    function (runState) {
+      const key = runState.stack.pop()
+      const keyBuf = setLengthLeft(bigIntToBytes(key), 32)
+      const value = runState.interpreter.transientStorageLoad(keyBuf)
+      const valueBN = value.length ? bytesToBigInt(value) : BIGINT_0
+      runState.stack.push(valueBN)
     },
   ],
-  // 0x5d: RETURNSUB (EIP 2315) / TSTORE (EIP 1153)
+  // 0x5d: TSTORE (EIP 1153)
   [
     0x5d,
-    function (runState, common) {
-      if (common.isActivatedEIP(2315)) {
-        // RETURNSUB
-        if (runState.returnStack.length < 1) {
-          trap(ERROR.INVALID_RETURNSUB)
-        }
-
-        const dest = runState.returnStack.pop()
-        runState.programCounter = Number(dest)
-      } else if (common.isActivatedEIP(1153)) {
-        // TSTORE
-        if (runState.interpreter.isStatic()) {
-          trap(ERROR.STATIC_STATE_CHANGE)
-        }
-        const [key, val] = runState.stack.popN(2)
-
-        const keyBuf = setLengthLeft(bigIntToBytes(key), 32)
-        // NOTE: this should be the shortest representation
-        let value
-        if (val === BIGINT_0) {
-          value = Uint8Array.from([])
-        } else {
-          value = bigIntToBytes(val)
-        }
+    function (runState) {
+      // TSTORE
+      if (runState.interpreter.isStatic()) {
+        trap(ERROR.STATIC_STATE_CHANGE)
+      }
+      const [key, val] = runState.stack.popN(2)
 
-        runState.interpreter.transientStorageStore(keyBuf, value)
+      const keyBuf = setLengthLeft(bigIntToBytes(key), 32)
+      // NOTE: this should be the shortest representation
+      let value
+      if (val === BIGINT_0) {
+        value = Uint8Array.from([])
+      } else {
+        value = bigIntToBytes(val)
       }
+
+      runState.interpreter.transientStorageStore(keyBuf, value)
     },
   ],
-  // 0x5e: JUMPSUB (2315) / MCOPY (5656)
+  // 0x5e: MCOPY (5656)
   [
     0x5e,
-    function (runState, common) {
-      if (common.isActivatedEIP(2315)) {
-        // JUMPSUB
-        const dest = runState.stack.pop()
-
-        if (dest > runState.interpreter.getCodeSize()) {
-          trap(ERROR.INVALID_JUMPSUB + ' at ' + describeLocation(runState))
-        }
-
-        const destNum = Number(dest)
-
-        if (!jumpSubIsValid(runState, destNum)) {
-          trap(ERROR.INVALID_JUMPSUB + ' at ' + describeLocation(runState))
-        }
-
-        runState.returnStack.push(BigInt(runState.programCounter))
-        runState.programCounter = destNum + 1
-      } else if (common.isActivatedEIP(5656)) {
-        // MCOPY
-        const [dst, src, length] = runState.stack.popN(3)
-        const data = runState.memory.read(Number(src), Number(length), true)
-        runState.memory.write(Number(dst), Number(length), data)
-      }
+    function (runState) {
+      const [dst, src, length] = runState.stack.popN(3)
+      const data = runState.memory.read(Number(src), Number(length), true)
+      runState.memory.write(Number(dst), Number(length), data)
     },
   ],
   // 0x5f: PUSH0

packages/evm/src/opcodes/util.ts

@@ -136,13 +136,6 @@ export function jumpIsValid(runState: RunState, dest: number): boolean {
   return runState.validJumps[dest] === 1
 }
 
-/**
- * Checks if a jumpsub is valid given a destination (defined as a 2 in the validJumps array)
- */
-export function jumpSubIsValid(runState: RunState, dest: number): boolean {
-  return runState.validJumps[dest] === 2
-}
-
 /**
  * Returns an overflow-safe slice of an array. It right-pads
  * the data with zeros to `length`.

packages/evm/src/types.ts

@@ -179,7 +179,6 @@ export interface EVMOpts {
    *
    * - [EIP-1153](https://eips.ethereum.org/EIPS/eip-1153) - Transient storage opcodes (Cancun)
    * - [EIP-1559](https://eips.ethereum.org/EIPS/eip-1559) - Fee market change for ETH 1.0 chain
-   * - [EIP-2315](https://eips.ethereum.org/EIPS/eip-2315) - Simple subroutines for the EVM (`outdated`)
    * - [EIP-2537](https://eips.ethereum.org/EIPS/eip-2537) - BLS precompiles (removed in v4.0.0, see latest v3 release)
    * - [EIP-2565](https://eips.ethereum.org/EIPS/eip-2565) - ModExp gas cost
    * - [EIP-2718](https://eips.ethereum.org/EIPS/eip-2565) - Transaction Types

packages/evm/test/opcodes.spec.ts

@@ -6,7 +6,6 @@ import { EVM } from '../src/index.js'
 describe('EVM -> getActiveOpcodes()', () => {
   const DIFFICULTY_PREVRANDAO = 0x44
   const CHAINID = 0x46 //istanbul opcode
-  const BEGINSUB = 0x5c // EIP-2315 opcode
 
   it('should not expose opcodes from a follow-up HF (istanbul -> petersburg)', async () => {
     const common = new Common({ chain: Chain.Mainnet, hardfork: Hardfork.Petersburg })
@@ -54,24 +53,6 @@ describe('EVM -> getActiveOpcodes()', () => {
     )
   })
 
-  it('should expose opcodes when EIP is active', async () => {
-    let common = new Common({ chain: Chain.Mainnet, hardfork: Hardfork.Istanbul, eips: [2315] })
-    let evm = await EVM.create({ common })
-    assert.equal(
-      evm.getActiveOpcodes().get(BEGINSUB)!.name,
-      'BEGINSUB',
-      'EIP-2315 opcode BEGINSUB exposed (EIP-2315 activated)'
-    )
-
-    common = new Common({ chain: Chain.Mainnet, hardfork: Hardfork.Istanbul })
-    evm = await EVM.create({ common })
-    assert.equal(
-      evm.getActiveOpcodes().get(BEGINSUB),
-      undefined,
-      'EIP-2315 opcode BEGINSUB not exposed (EIP-2315 not activated)'
-    )
-  })
-
   it('should update opcodes on a hardfork change', async () => {
     const common = new Common({ chain: Chain.Mainnet, hardfork: Hardfork.Istanbul })
     const evm = await EVM.create({ common })