LibJS: Store ECMAScriptFunctionObject bytecode in an OwnPtr

Using an Optional was extremely wasteful for function objects that don't
even have a bytecode executable.

This allows ECMAScriptFunctionObject to fit in a smaller size class.

1 files changed, 10 insertions, 10 deletions

diff --git a/Tests/LibJS/test-bytecode-js.cpp b/Tests/LibJS/test-bytecode-js.cpp
index e2dad31beb..267d2a7e62 100644
--- a/Tests/LibJS/test-bytecode-js.cpp
+++ b/Tests/LibJS/test-bytecode-js.cpp
@@ -24,17 +24,17 @@
 
 #define EXPECT_NO_EXCEPTION(executable)                           \
     auto executable = JS::Bytecode::Generator::generate(program); \
-    auto result = bytecode_interpreter.run(executable);           \
+    auto result = bytecode_interpreter.run(*executable);          \
     EXPECT(!result.is_error());                                   \
     EXPECT(!vm->exception());
 
-#define EXPECT_NO_EXCEPTION_WITH_OPTIMIZATIONS(executable)                 \
-    auto& passes = JS::Bytecode::Interpreter::optimization_pipeline();     \
-    passes.perform(executable);                                            \
-                                                                           \
-    auto result_with_optimizations = bytecode_interpreter.run(executable); \
-                                                                           \
-    EXPECT(!result_with_optimizations.is_error());                         \
+#define EXPECT_NO_EXCEPTION_WITH_OPTIMIZATIONS(executable)                  \
+    auto& passes = JS::Bytecode::Interpreter::optimization_pipeline();      \
+    passes.perform(*executable);                                            \
+                                                                            \
+    auto result_with_optimizations = bytecode_interpreter.run(*executable); \
+                                                                            \
+    EXPECT(!result_with_optimizations.is_error());                          \
     EXPECT(!vm->exception())
 
 #define EXPECT_NO_EXCEPTION_ALL(source) \
@@ -57,7 +57,7 @@ TEST_CASE(if_statement_fail)
     SETUP_AND_PARSE("if (true) throw new Exception('failed');");
 
     auto executable = JS::Bytecode::Generator::generate(program);
-    auto result = bytecode_interpreter.run(executable);
+    auto result = bytecode_interpreter.run(*executable);
     EXPECT(result.is_error());
 }
 
@@ -115,7 +115,7 @@ TEST_CASE(loading_multiple_files)
         auto const& test_file_program = test_file_script->parse_node();
 
         auto executable = JS::Bytecode::Generator::generate(test_file_program);
-        auto result = bytecode_interpreter.run(executable);
+        auto result = bytecode_interpreter.run(*executable);
         EXPECT(!result.is_error());
         EXPECT(!vm->exception());
     }