From 9a709e03a314d9b8c34d2c0c7f1806b4aa152069 Mon Sep 17 00:00:00 2001 From: Andreas Kling Date: Mon, 7 Aug 2023 16:47:04 +0200 Subject: [PATCH] test262-runner: Remove ability to run with AST interpreter I'm leaving the --use-bytecode CLI option here as a no-op for now, until we get all the scripts updated. But the program always runs in bytecode mode now. --- Tests/LibJS/test262-runner.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Tests/LibJS/test262-runner.cpp b/Tests/LibJS/test262-runner.cpp index 148abe7db08..56a63d45ff7 100644 --- a/Tests/LibJS/test262-runner.cpp +++ b/Tests/LibJS/test262-runner.cpp @@ -567,6 +567,7 @@ int main(int argc, char** argv) Core::ArgsParser args_parser; args_parser.set_general_help("LibJS test262 runner for streaming tests"); args_parser.add_option(s_harness_file_directory, "Directory containing the harness files", "harness-location", 'l', "harness-files"); + // FIXME: Remove this option after updating the test262 scripts. args_parser.add_option(use_bytecode, "Use the bytecode interpreter", "use-bytecode", 'b'); args_parser.add_option(s_parse_only, "Only parse the files", "parse-only", 'p'); args_parser.add_option(timeout, "Seconds before test should timeout", "timeout", 't', "seconds"); @@ -574,7 +575,7 @@ 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(use_bytecode); + 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) {