diff options
| author | Simon Hausmann <simon.hausmann@digia.com> | 2013-10-18 14:02:56 +0200 |
|---|---|---|
| committer | The Qt Project <gerrit-noreply@qt-project.org> | 2013-10-20 21:11:43 +0200 |
| commit | 6b2b62e903e1207255b0652b728ecaee6d51aea9 (patch) | |
| tree | c832e832680f9d9fe21fc3eda8f7589efd78362a /src/qml/compiler/qv4jsir.cpp | |
| parent | 9fd4591a61df467362fa2f17cc2d506efd6bece3 (diff) | |
Cleanup: Avoid unnecessary irModule->functions.indexOf calls
With a growing number of functions per module, these calls become
expensive and are unnecessary. defineFunction in the code generator
can simply return the correct index right away.
Change-Id: I8ab56a3083bf215674a1b46c502b415be694e465
Reviewed-by: Lars Knoll <lars.knoll@digia.com>
Diffstat (limited to 'src/qml/compiler/qv4jsir.cpp')
| -rw-r--r-- | src/qml/compiler/qv4jsir.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/qml/compiler/qv4jsir.cpp b/src/qml/compiler/qv4jsir.cpp index 756c076abf..869bf4acaf 100644 --- a/src/qml/compiler/qv4jsir.cpp +++ b/src/qml/compiler/qv4jsir.cpp @@ -464,7 +464,7 @@ bool operator<(const Temp &t1, const Temp &t2) Q_DECL_NOTHROW void Closure::dump(QTextStream &out) const { - QString name = value->name ? *value->name : QString(); + QString name = functionName ? *functionName : QString(); if (name.isEmpty()) name.sprintf("%p", value); out << "closure(" << name << ')'; @@ -763,10 +763,10 @@ Name *BasicBlock::NAME(Name::Builtin builtin, quint32 line, quint32 column) return e; } -Closure *BasicBlock::CLOSURE(Function *function) +Closure *BasicBlock::CLOSURE(int functionInModule) { Closure *clos = function->New<Closure>(); - clos->init(function); + clos->init(functionInModule, function->module->functions.at(functionInModule)->name); return clos; } |