Simplify storeProperty runtime API

We can do the isStrict() check and throwTypeError() call in the runtime
function instead of doing it twice on the two call sites.

Change-Id: I5689babe4873c0bc3e7c85ca2d15a33f8ccb8794
Reviewed-by: Lars Knoll <lars.knoll@qt.io>

1 files changed, 2 insertions, 9 deletions

diff --git a/src/qml/jit/qv4baselinejit.cpp b/src/qml/jit/qv4baselinejit.cpp
index 9046926196..ed4434d4a0 100644
--- a/src/qml/jit/qv4baselinejit.cpp
+++ b/src/qml/jit/qv4baselinejit.cpp
@@ -301,13 +301,6 @@ void BaselineJIT::generate_GetLookupA(int index)
     as->checkException();
 }
 
-static void storePropertyHelper(QV4::Function *f, const Value &base, int name, const Value &value)
-{
-    auto engine = f->internalClass->engine;
-    if (!Runtime::method_storeProperty(engine, base, name, value) && f->isStrict())
-        engine->throwTypeError();
-}
-
 void BaselineJIT::generate_StoreProperty(int name, int base)
 {
     STORE_IP();
@@ -316,8 +309,8 @@ void BaselineJIT::generate_StoreProperty(int name, int base)
     as->passAccumulatorAsArg(3);
     as->passInt32AsArg(name, 2);
     as->passRegAsArg(base, 1);
-    as->passFunctionAsArg(0);
-    JIT_GENERATE_RUNTIME_CALL(storePropertyHelper, Assembler::IgnoreResult);
+    as->passEngineAsArg(0);
+    JIT_GENERATE_RUNTIME_CALL(Runtime::method_storeProperty, Assembler::IgnoreResult);
     as->checkException();
 }