Attachment 340295 Details for Bug 185601 – Patch

[patch] Patch

bug-185601-20180514160949.patch (text/plain), 3.49 KB, created by Yusuke Suzuki on 2018-05-14 00:09:50 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Yusuke Suzuki

Created: 2018-05-14 00:09:50 PDT

Size: 3.49 KB

patch

obsolete

>Subversion Revision: 231744
>diff --git a/Source/JavaScriptCore/ChangeLog b/Source/JavaScriptCore/ChangeLog
>index 9236b46544dfd53012c6077d2bb79c3f681035a6..e226386e5dc3a17751425060909a91c29935a06a 100644
>--- a/Source/JavaScriptCore/ChangeLog
>+++ b/Source/JavaScriptCore/ChangeLog
>@@ -1,3 +1,28 @@
>+2018-05-14  Yusuke Suzuki  <utatane.tea@gmail.com>
>+
>+        [JSC] Check TypeOfShouldCallGetCallData before calling getCallData when we would like to check whether a given object is callable
>+        https://bugs.webkit.org/show_bug.cgi?id=185601
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Check TypeOfShouldCallGetCallData before calling getCallData when we would like to check
>+        whether a given object is callable since getCallData is a virtual call. When we call
>+        the object anyway, directly calling getCallData is fine. But if we would like to check
>+        whether the object is callable, we can have non callable objects frequently. In that case,
>+        we should not call getCallData if we can avoid it.
>+
>+        We found that this virtual call exists in JSON.stringify's critial path. Checking
>+        TypeOfShouldCallGetCallData improves Kraken/json-stringify-tinderbox by 2-3%.
>+
>+                                               baseline                  patched
>+
>+            json-stringify-tinderbox        39.053+-0.730      ^      37.382+-0.351         ^ definitely 1.0447x faster
>+
>+        * runtime/JSONObject.cpp:
>+        (JSC::Stringifier::appendStringifiedValue):
>+        * runtime/Operations.cpp:
>+        (JSC::jsIsObjectTypeOrNull):
>+
> 2018-05-12  Filip Pizlo  <fpizlo@apple.com>
> 
>         CachedCall::call() should be faster
>diff --git a/Source/JavaScriptCore/runtime/JSONObject.cpp b/Source/JavaScriptCore/runtime/JSONObject.cpp
>index b3a7808b64fe36a86a28134fe79ab1ddcdfd3a01..38ef04348228a8327c36d03d1786e9734000d7db 100644
>--- a/Source/JavaScriptCore/runtime/JSONObject.cpp
>+++ b/Source/JavaScriptCore/runtime/JSONObject.cpp
>@@ -382,7 +382,8 @@ Stringifier::StringifyResult Stringifier::appendStringifiedValue(StringBuilder&
>     JSObject* object = asObject(value);
> 
>     CallData callData;
>-    if (object->methodTable(vm)->getCallData(object, callData) != CallType::None) {
>+    if (object->inherits<JSFunction>(vm)
>+        || (object->inlineTypeFlags() & TypeOfShouldCallGetCallData && object->methodTable(vm)->getCallData(object, callData) != CallType::None)) {
>         if (holder.isArray()) {
>             builder.appendLiteral("null");
>             return StringifySucceeded;
>diff --git a/Source/JavaScriptCore/runtime/Operations.cpp b/Source/JavaScriptCore/runtime/Operations.cpp
>index 97cfa07a9694022256c9c38a66c76433517cd65e..ab37895ece5bd6b9511d9e0a4ad1f226cc312822 100644
>--- a/Source/JavaScriptCore/runtime/Operations.cpp
>+++ b/Source/JavaScriptCore/runtime/Operations.cpp
>@@ -119,10 +119,14 @@ bool jsIsObjectTypeOrNull(CallFrame* callFrame, JSValue v)
>     if (type >= ObjectType) {
>         if (asObject(v)->structure(vm)->masqueradesAsUndefined(callFrame->lexicalGlobalObject()))
>             return false;
>-        CallData callData;
>         JSObject* object = asObject(v);
>-        if (object->methodTable(vm)->getCallData(object, callData) != CallType::None)
>+        if (object->inherits<JSFunction>(vm))
>             return false;
>+        if (object->inlineTypeFlags() & TypeOfShouldCallGetCallData) {
>+            CallData callData;
>+            if (object->methodTable(vm)->getCallData(object, callData) != CallType::None)
>+                return false;
>+        }
>     }
>     return true;
> }

Actions: View | Formatted Diff | Diff

Attachments on bug 185601: 340290 | 340291 | 340292 | 340294 | 340295 | 340323 | 340329 | 340331 | 340333 | 340393