Remove IR::Function from the runtime.

This fixes potential leaks of IR::Functions, lowers the memory usage
of the functions that the VM needs (because the IR fields are not
present in the VM::Function), and makes both managed by the module
respectively the ExecutionEngine.

Change-Id: I6748ad98b062f994eae9dd14f1919aec5aa7c0b0
Reviewed-by: Lars Knoll <lars.knoll@digia.com>

1 files changed, 5 insertions, 1 deletions

diff --git a/qmljs_engine.h b/qmljs_engine.h
index 1b4e593200..eef62a3249 100644
--- a/qmljs_engine.h
+++ b/qmljs_engine.h
@@ -56,6 +56,7 @@ namespace VM {
 
 struct Value;
 class Array;
+struct Function;
 struct Object;
 struct BooleanObject;
 struct NumberObject;
@@ -156,6 +157,7 @@ struct ExecutionEngine
     Value exception;
 
     struct StringPool *stringPool;
+    QVector<Function *> functions;
 
     ExecutionEngine(MemoryManager *memoryManager, EvalISelFactory *iselFactory);
     ~ExecutionEngine();
@@ -164,8 +166,10 @@ struct ExecutionEngine
 
     String *identifier(const QString &s);
 
+    VM::Function *newFunction(const QString &name);
+
     FunctionObject *newNativeFunction(ExecutionContext *scope, String *name, Value (*code)(ExecutionContext *));
-    FunctionObject *newScriptFunction(ExecutionContext *scope, IR::Function *function);
+    FunctionObject *newScriptFunction(ExecutionContext *scope, VM::Function *function);
 
     Object *newObject();
     FunctionObject *newObjectCtor(ExecutionContext *ctx);