LibJS: Remove ability to disable bytecode VM

Author: https://github.com/awesomekling
Commit: https://github.com/SerenityOS/serenity/commit/e2c8d5859e
Pull-request: https://github.com/SerenityOS/serenity/pull/20421

Tests/LibJS/test-js.cpp

@@ -14,7 +14,8 @@ TESTJS_PROGRAM_FLAG(test262_parser_tests, "Run test262 parser tests", "test262-p
 
 TESTJS_GLOBAL_FUNCTION(is_bytecode_interpreter_enabled, isBytecodeInterpreterEnabled, 0)
 {
-    return JS::Value(JS::Bytecode::Interpreter::enabled());
+    // FIXME: Remove this function after updating the tests.
+    return JS::Value(true);
 }
 
 TESTJS_GLOBAL_FUNCTION(is_strict_mode, isStrictMode, 0)

Tests/LibJS/test262-runner.cpp

@@ -575,8 +575,6 @@ int main(int argc, char** argv)
     args_parser.add_option(disable_core_dumping, "Disable core dumping", "disable-core-dump", 0);
     args_parser.parse(arguments);
 
-    JS::Bytecode::Interpreter::set_enabled(true);
-
 #if !defined(AK_OS_MACOS) && !defined(AK_OS_EMSCRIPTEN)
     if (disable_core_dumping && prctl(PR_SET_DUMPABLE, 0, 0) < 0) {
         perror("prctl(PR_SET_DUMPABLE)");

Userland/Libraries/LibJS/Bytecode/Interpreter.cpp

@@ -19,18 +19,6 @@
 
 namespace JS::Bytecode {
 
-static bool s_bytecode_interpreter_enabled = false;
-
-bool Interpreter::enabled()
-{
-    return s_bytecode_interpreter_enabled;
-}
-
-void Interpreter::set_enabled(bool enabled)
-{
-    s_bytecode_interpreter_enabled = enabled;
-}
-
 bool g_dump_bytecode = false;
 
 Interpreter::Interpreter(VM& vm)

Userland/Libraries/LibJS/Bytecode/Interpreter.h

@@ -36,9 +36,6 @@ struct CallFrame {
 
 class Interpreter {
 public:
-    [[nodiscard]] static bool enabled();
-    static void set_enabled(bool);
-
     explicit Interpreter(VM&);
     ~Interpreter();
 

Userland/Libraries/LibJS/Runtime/VM.cpp

@@ -208,8 +208,6 @@ Bytecode::Interpreter& VM::bytecode_interpreter()
 
 Bytecode::Interpreter* VM::bytecode_interpreter_if_exists()
 {
-    if (!Bytecode::Interpreter::enabled())
-        return nullptr;
     return m_bytecode_interpreter;
 }
 

Userland/Libraries/LibTest/JavaScriptTestRunnerMain.cpp

@@ -134,8 +134,6 @@ int main(int argc, char** argv)
         AK::set_debug_enabled(false);
     }
 
-    JS::Bytecode::Interpreter::set_enabled(true);
-
     DeprecatedString test_root;
 
     if (!specified_test_root.is_empty()) {

Userland/Utilities/js.cpp

@@ -576,8 +576,6 @@ ErrorOr<int> serenity_main(Main::Arguments arguments)
     args_parser.add_positional_argument(script_paths, "Path to script files", "scripts", Core::ArgsParser::Required::No);
     args_parser.parse(arguments);
 
-    JS::Bytecode::Interpreter::set_enabled(true);
-
     bool syntax_highlight = !disable_syntax_highlight;
 
     AK::set_debug_enabled(!disable_debug_printing);