Clean up logic to make code more readable

Author: lrettig

wasm/generateInterface.js

@@ -333,11 +333,7 @@ function generateManifest (interfaceManifest, opts) {
         locals += `(local $offset${numOfLocals} i32)`
         body += `(set_local $offset${numOfLocals} ${checkOverflowStackItem256(spOffset)})`
         call += `(get_local $offset${numOfLocals})`
-      } else if (input === 'length' && (opcode === 'CALL' || opcode === 'CALLCODE' || opcode === 'DELEGATECALL' || opcode === 'STATICCALL')) {
-        // CALLs in EVM have 7 arguments
-        // but in ewasm CALLs only have 5 arguments
-        // so delete the bottom two stack elements, after processing the 5th argument
-
+      } else if (input === 'length') {
         locals += `(local $length${numOfLocals} i32)`
         body += `(set_local $length${numOfLocals} ${checkOverflowStackItem256(spOffset)})`
 
@@ -348,21 +344,13 @@ function generateManifest (interfaceManifest, opts) {
         call += `(get_local $length${numOfLocals})`
         numOfLocals++
 
-        // delete 6th stack element
-        spOffset--
-
-        // delete 7th stack element
-        spOffset--
-      } else if (input === 'length' && (opcode !== 'CALL' && opcode !== 'CALLCODE' && opcode !== 'DELEGATECALL' && opcode !== 'STATICCALL')) {
-        locals += `(local $length${numOfLocals} i32)`
-        body += `(set_local $length${numOfLocals} ${checkOverflowStackItem256(spOffset)})`
-
-        body += `
-    (call $memusegas (get_local $offset${numOfLocals}) (get_local $length${numOfLocals}))
-    (set_local $offset${numOfLocals} (i32.add (get_global $memstart) (get_local $offset${numOfLocals})))`
-
-        call += `(get_local $length${numOfLocals})`
-        numOfLocals++
+        // CALLs in EVM have 7 arguments
+        // but in ewasm CALLs only have 5 arguments
+        // so delete the bottom two stack elements, after processing the 5th argument
+        if (opcode === 'CALL' || opcode === 'CALLCODE') {
+          spOffset--
+          spOffset--
+        }
       }
       spOffset--
     })
@@ -408,21 +396,16 @@ function generateManifest (interfaceManifest, opts) {
         call += '(get_local $callback)'
       }
 
-      if (opcode === 'CALL' || opcode === 'CALLCODE' || opcode === 'DELEGATECALL' || opcode === 'STATICCALL') {
-        call =
-          `(i64.store
-      (i32.add (get_global $sp) (i32.const ${spOffset * 32}))
-      (i64.extend_u/i32
-        (i32.eqz ${call}) ;; flip CALL result from EEI to EVM convention (0 -> 1, 1,2,.. -> 1)
-      )))`
-
+      let wrapper
+      if (opcode === 'CALL' || opcode === 'CALLCODE' || opcode === 'DELEGATECALL') {
+        // flip CALL result from EEI to EVM convention (0 -> 1, 1,2,.. -> 0)
+        wrapper = `(i32.eqz ${call})`
       } else {
-        call =
-          `(i64.store
-      (i32.add (get_global $sp) (i32.const ${spOffset * 32}))
-      (i64.extend_u/i32
-        ${call})))`
+        wrapper = call
       }
+      call = `(i64.store
+    (i32.add (get_global $sp) (i32.const ${spOffset * 32}))
+    (i64.extend_u/i32 ${wrapper})))`
 
       call += cleanupStackItem64(spOffset)
     } else if (output === 'i64') {