Replace O=2 option with advanced

ethereum · Aug 26, 2022 · ed346f9 · ed346f9
1 parent f623b68
commit ed346f9
Show file tree

Hide file tree

Showing 7 changed files with 19 additions and 33 deletions.
diff --git a/lib/evmone/vm.cpp b/lib/evmone/vm.cpp
@@ -33,19 +33,10 @@ evmc_set_option_result set_option(evmc_vm* c_vm, char const* c_name, char const*
     const auto value = (c_value != nullptr) ? std::string_view{c_value} : std::string_view{};
     auto& vm = *static_cast<VM*>(c_vm);
 
-    if (name == "O")
+    if (name == "advanced")
     {
-        if (value == "0")
-        {
-            c_vm->execute = evmone::baseline::execute;
-            return EVMC_SET_OPTION_SUCCESS;
-        }
-        else if (value == "2")
-        {
-            c_vm->execute = evmone::advanced::execute;
-            return EVMC_SET_OPTION_SUCCESS;
-        }
-        return EVMC_SET_OPTION_INVALID_VALUE;
+        c_vm->execute = evmone::advanced::execute;
+        return EVMC_SET_OPTION_SUCCESS;
     }
     else if (name == "cgoto")
     {

diff --git a/test/bench/bench.cpp b/test/bench/bench.cpp
@@ -317,9 +317,9 @@ int main(int argc, char** argv)
         if (ec != 0)
             return ec;
 
-        registered_vms["advanced"] = evmc::VM{evmc_create_evmone(), {{"O", "2"}}};
-        registered_vms["baseline"] = evmc::VM{evmc_create_evmone(), {{"O", "0"}}};
-        registered_vms["bnocgoto"] = evmc::VM{evmc_create_evmone(), {{"O", "0"}, {"cgoto", "no"}}};
+        registered_vms["advanced"] = evmc::VM{evmc_create_evmone(), {{"advanced", ""}}};
+        registered_vms["baseline"] = evmc::VM{evmc_create_evmone()};
+        registered_vms["bnocgoto"] = evmc::VM{evmc_create_evmone(), {{"cgoto", "no"}}};
         register_benchmarks(benchmark_cases);
         register_synthetic_benchmarks();
         RunSpecifiedBenchmarks();

diff --git a/test/fuzzer/fuzzer.cpp b/test/fuzzer/fuzzer.cpp
@@ -55,10 +55,10 @@ template <typename T1, typename T2>
 static auto print_input = std::getenv("PRINT");
 
 /// The reference VM: evmone Baseline
-static auto ref_vm = evmc::VM{evmc_create_evmone(), {{"O", "0"}}};
+static auto ref_vm = evmc::VM{evmc_create_evmone()};
 
 static evmc::VM external_vms[] = {
-    evmc::VM{evmc_create_evmone(), {{"O", "2"}}},
+    evmc::VM{evmc_create_evmone(), {{"advanced", ""}}},
 };
 
 

diff --git a/test/integration/CMakeLists.txt b/test/integration/CMakeLists.txt
@@ -7,7 +7,7 @@ set(PREFIX ${PROJECT_NAME}/integration)
 get_target_property(EVMONE_LIB_TYPE evmone TYPE)
 if(EVMONE_LIB_TYPE STREQUAL SHARED_LIBRARY)
 
-    add_test(NAME ${PREFIX}/histogram COMMAND $<TARGET_FILE:evmc::tool> --vm $<TARGET_FILE:evmone>,O=0,histogram run 6000808080800101010200)
+    add_test(NAME ${PREFIX}/histogram COMMAND $<TARGET_FILE:evmc::tool> --vm $<TARGET_FILE:evmone>,histogram run 6000808080800101010200)
     set_tests_properties(
         ${PREFIX}/histogram PROPERTIES PASS_REGULAR_EXPRESSION
         "--- # HISTOGRAM depth=0
@@ -19,7 +19,7 @@ PUSH1,1
 DUP1,4
 ")
 
-    add_test(NAME ${PREFIX}/trace COMMAND $<TARGET_FILE:evmc::tool> --vm $<TARGET_FILE:evmone>,O=0,trace run 60006002800103)
+    add_test(NAME ${PREFIX}/trace COMMAND $<TARGET_FILE:evmc::tool> --vm $<TARGET_FILE:evmone>,trace run 60006002800103)
     set_tests_properties(
         ${PREFIX}/trace PROPERTIES PASS_REGULAR_EXPRESSION
         "{\"depth\":0,\"rev\":\"London\",\"static\":false}

diff --git a/test/unittests/evm_fixture.cpp b/test/unittests/evm_fixture.cpp
@@ -9,9 +9,9 @@ namespace evmone::test
 {
 namespace
 {
-evmc::VM advanced_vm{evmc_create_evmone(), {{"O", "2"}}};
-evmc::VM baseline_vm{evmc_create_evmone(), {{"O", "0"}}};
-evmc::VM bnocgoto_vm{evmc_create_evmone(), {{"O", "0"}, {"cgoto", "no"}}};
+evmc::VM advanced_vm{evmc_create_evmone(), {{"advanced", ""}}};
+evmc::VM baseline_vm{evmc_create_evmone()};
+evmc::VM bnocgoto_vm{evmc_create_evmone(), {{"cgoto", "no"}}};
 
 const char* print_vm_name(const testing::TestParamInfo<evmc::VM*>& info) noexcept
 {

diff --git a/test/unittests/evmone_test.cpp b/test/unittests/evmone_test.cpp
@@ -32,18 +32,13 @@ TEST(evmone, set_option_invalid)
     vm->destroy(vm);
 }
 
-TEST(evmone, set_option_optimization_level)
+TEST(evmone, set_option_advanced)
 {
     auto vm = evmc::VM{evmc_create_evmone()};
-    EXPECT_EQ(vm.set_option("O", ""), EVMC_SET_OPTION_INVALID_VALUE);
-    EXPECT_EQ(vm.set_option("O", "0"), EVMC_SET_OPTION_SUCCESS);
-    EXPECT_EQ(vm.set_option("O", "1"), EVMC_SET_OPTION_INVALID_VALUE);
-    EXPECT_EQ(vm.set_option("O", "2"), EVMC_SET_OPTION_SUCCESS);
-    EXPECT_EQ(vm.set_option("O", "3"), EVMC_SET_OPTION_INVALID_VALUE);
-
-    EXPECT_EQ(vm.set_option("O", "20"), EVMC_SET_OPTION_INVALID_VALUE);
-    EXPECT_EQ(vm.set_option("O", "21"), EVMC_SET_OPTION_INVALID_VALUE);
-    EXPECT_EQ(vm.set_option("O", "22"), EVMC_SET_OPTION_INVALID_VALUE);
+    EXPECT_EQ(vm.set_option("advanced", ""), EVMC_SET_OPTION_SUCCESS);
+
+    // This will also enable Advanced.
+    EXPECT_EQ(vm.set_option("advanced", "no"), EVMC_SET_OPTION_SUCCESS);
 }
 
 TEST(evmone, set_option_cgoto)

diff --git a/test/unittests/tracing_test.cpp b/test/unittests/tracing_test.cpp
@@ -23,7 +23,7 @@ class tracing : public Test
     std::ostringstream trace_stream;
 
     tracing()
-      : m_baseline_vm{evmc_create_evmone(), {{"O", "0"}}},
+      : m_baseline_vm{evmc_create_evmone()},
         vm{*static_cast<evmone::VM*>(m_baseline_vm.get_raw_pointer())}
     {}