Merge pull request #573 from wasmx/fix_invoke

Fix invoke: drop args after the call

Author: axic

lib/fizzy/execute.cpp

@@ -426,19 +426,20 @@ void branch(const Code& code, OperandStack& stack, const Instr*& pc, const uint8
 }
 
 template <class F>
-bool invoke_function(const FuncType& func_type, const F& func, Instance& instance,
+inline bool invoke_function(const FuncType& func_type, const F& func, Instance& instance,
     OperandStack& stack, int depth) noexcept
 {
     const auto num_args = func_type.inputs.size();
     assert(stack.size() >= num_args);
     span<const Value> call_args{stack.rend() - num_args, num_args};
-    stack.drop(num_args);
 
     const auto ret = func(instance, call_args, depth + 1);
     // Bubble up traps
     if (ret.trapped)
         return false;
 
+    stack.drop(num_args);
+
     const auto num_outputs = func_type.outputs.size();
     // NOTE: we can assume these two from validation
     assert(num_outputs <= 1);