EVM: Refactor Interpreter dispatch (#646)

* check stack limits at dispatch, make stack ops unchecked

* use a growable vector for the stack

* fix growth check

* clippy wants default...

* optimize ensure somewhat

* remove implicit assumption about growth patterns

* drop error from Bytecode::new signature, it can't really err.

fix rebase artifact

* refactor part I: primops

* add arity/stack checks for primops

* dup and swap

* push

* immediates

* sugar

* first stdfun

* keccak256 as stdfun and prettier sugar

* moar stdfuns

* calldatacopy

* emacs auto-indent friendly sugar

* more functionoids

* moar funcionoids

* call opcodes

also kill leftover callactor

* primitive -> primop

* codesize and codecopy

* create and create2

* control flow

* mark stack ops as unsafe

only macro mechanically checked invocations for these

* fix tests

* rustfmt

* shut up clippy's frivolous complaints

* nit: macro magic

Co-authored-by: Steven Allen <steven@stebalien.com>

Author: vyzo

actors/evm/src/interpreter/bytecode.rs

@@ -12,7 +12,6 @@ impl Bytecode {
     pub fn new(bytecode: Vec<u8>) -> Self {
         // only jumps to those addresses are valid. This is a security
         // feature by EVM to disallow jumps to arbitary code addresses.
-        // todo: create the jumpdest table only once during initial contract deployment
         let mut jumpdest = vec![false; bytecode.len()];
         let mut i = 0;
         while i < bytecode.len() {