Move execute(Module, ...) to execute_helpers.hpp

Author: chfast

lib/fizzy/execute.cpp

@@ -1464,12 +1464,6 @@ execution_result execute(
     return {trap, {stack.rbegin(), stack.rend()}};
 }
 
-execution_result execute(const Module& module, FuncIdx func_idx, std::vector<uint64_t> args)
-{
-    auto instance = instantiate(module);
-    return execute(*instance, func_idx, std::move(args));
-}
-
 std::vector<ExternalFunction> resolve_imported_functions(
     const Module& module, std::vector<ImportedFunction> imported_functions)
 {

lib/fizzy/execute.hpp

@@ -97,8 +97,6 @@ std::unique_ptr<Instance> instantiate(Module module,
 execution_result execute(
     Instance& instance, FuncIdx func_idx, std::vector<uint64_t> args, int depth = 0);
 
-// TODO: remove this helper
-execution_result execute(const Module& module, FuncIdx func_idx, std::vector<uint64_t> args);
 
 // Function that should be used by instantiate as imports, identified by module and function name.
 struct ImportedFunction

test/unittests/execute_call_test.cpp

@@ -7,6 +7,7 @@
 #include "parser.hpp"
 #include <gtest/gtest.h>
 #include <test/utils/asserts.hpp>
+#include <test/utils/execute_helpers.hpp>
 #include <test/utils/hex.hpp>
 
 using namespace fizzy;

test/unittests/execute_control_test.cpp

@@ -6,6 +6,7 @@
 #include "parser.hpp"
 #include <gtest/gtest.h>
 #include <test/utils/asserts.hpp>
+#include <test/utils/execute_helpers.hpp>
 #include <test/utils/hex.hpp>
 
 using namespace fizzy;

test/unittests/execute_numeric_test.cpp

@@ -6,6 +6,7 @@
 #include "parser.hpp"
 #include <gtest/gtest.h>
 #include <test/utils/asserts.hpp>
+#include <test/utils/execute_helpers.hpp>
 #include <test/utils/hex.hpp>
 
 using namespace fizzy;

test/unittests/execute_test.cpp

@@ -7,6 +7,7 @@
 #include "parser.hpp"
 #include <gtest/gtest.h>
 #include <test/utils/asserts.hpp>
+#include <test/utils/execute_helpers.hpp>
 #include <test/utils/hex.hpp>
 
 using namespace fizzy;

test/utils/CMakeLists.txt

@@ -13,6 +13,7 @@ target_sources(
     adler32.hpp
     asserts.cpp
     asserts.hpp
+    execute_helpers.hpp
     fizzy_engine.cpp
     hex.cpp
     hex.hpp

test/utils/execute_helpers.hpp

@@ -0,0 +1,16 @@
+// Fizzy: A fast WebAssembly interpreter
+// Copyright 2020 The Fizzy Authors.
+// SPDX-License-Identifier: Apache-2.0
+
+#pragma once
+
+#include "execute.hpp"
+
+namespace fizzy::test
+{
+inline execution_result execute(const Module& module, FuncIdx func_idx, std::vector<uint64_t> args)
+{
+    auto instance = instantiate(module);
+    return execute(*instance, func_idx, std::move(args));
+}
+}  // namespace fizzy::test