[1.7>master] [MERGE #3751 @obastemur] perf: Improve JSON.stringify performance

Merge pull request #3751 from obastemur:fjs

Kraken/json-stringify-* perf ~25% better. Acme Air - LTO gain ~1.5%

PR Details:

- Improve `replacer != function && !HasObjectArray` case. (most common use of JSON.stringify)

- Add JSON.stringify test cases for ObjectArray, toJSON, and replacer function

- IsNumericPropertyId: fast path for internal properties

- use requestContext instead of instance->scriptContext

- use wmemcpy instead of memcpy

Author: obastemur

.gitignore

@@ -92,6 +92,7 @@ tags
 Makefile
 pal/src/config.h
 DbgController.js.h
+lib/wabt/built/config.h
 
 # Generated by other tools
 *.lldb.cmd

lib/Common/Core/Api.cpp

@@ -3,6 +3,9 @@
 // Licensed under the MIT license. See LICENSE.txt file in the project root for full license information.
 //-------------------------------------------------------------------------------------------------------
 #include "CommonCorePch.h"
+#ifdef _WIN32
+#include <wchar.h> // wmemcpy_s
+#endif
 
 void __stdcall js_memcpy_s(__bcount(sizeInBytes) void *dst, size_t sizeInBytes, __in_bcount(count) const void *src, size_t count)
 {
@@ -21,8 +24,7 @@ void __stdcall js_wmemcpy_s(__ecount(sizeInWords) char16 *dst, size_t sizeInWord
     {
         Js::Throw::FatalInternalError();
     }
-
-    memcpy(dst, src, count * sizeof(char16));
+    wmemcpy_s(dst, count, src, count);
 }
 
 #if defined(_M_IX86) || defined(_M_X64)

lib/Runtime/Base/ThreadContext.cpp

@@ -1244,6 +1244,11 @@ void ThreadContext::AddBuiltInPropertyRecord(const Js::PropertyRecord *propertyR
 
 BOOL ThreadContext::IsNumericPropertyId(Js::PropertyId propertyId, uint32* value)
 {
+    if (Js::IsInternalPropertyId(propertyId))
+    {
+        return false;
+    }
+
     Js::PropertyRecord const * propertyRecord = this->GetPropertyName(propertyId);
     Assert(propertyRecord != nullptr);
     if (propertyRecord == nullptr || !propertyRecord->IsNumeric())

lib/Runtime/Library/JSON.cpp

@@ -61,7 +61,7 @@ namespace JSON
         }
         void CompleteInit(Js::Var space, ArenaAllocator* alloc);
 
-        Js::Var Str(Js::JavascriptString* key, Js::PropertyId keyId, Js::Var holder);
+        Js::Var Str(Js::JavascriptString* key, Js::PropertyId keyId, Js::Var holder, Js::Var value = nullptr);
         Js::Var Str(uint32 index, Js::Var holder);
 
     private:
@@ -74,8 +74,10 @@ namespace JSON
         Js::JavascriptString* GetPropertySeparator();
         Js::JavascriptString* GetIndentString(uint count);
         Js::JavascriptString* GetMemberSeparator(Js::JavascriptString* indentString);
-        void StringifyMemberObject( Js::JavascriptString* propertyName, Js::PropertyId id, Js::Var value, Js::ConcatStringBuilder* result,
-            Js::JavascriptString* &indentString, Js::JavascriptString* &memberSeparator, bool &isFirstMember, bool &isEmpty );
+        void StringifyMemberObject(Js::JavascriptString* propertyName, Js::PropertyId id,
+          Js::Var value, Js::ConcatStringBuilder* result, Js::JavascriptString* &indentString,
+          Js::JavascriptString* &memberSeparator, bool &isFirstMember, bool &isEmpty,
+          Js::Var propertyValue = nullptr );
 
         uint32 GetPropertyCount(Js::RecyclableObject* object, Js::JavascriptStaticEnumerator* enumerator);
         uint32 GetPropertyCount(Js::RecyclableObject* object, Js::JavascriptStaticEnumerator* enumerator, bool* isPrecise);

lib/Runtime/Library/JSON.h

@@ -121,7 +121,6 @@ namespace Js
         // For boxing stack instance
         DynamicObject(DynamicObject * instance);
 
-        DynamicTypeHandler * GetTypeHandler() const;
         uint16 GetOffsetOfInlineSlots() const;
 
         template <class T>
@@ -136,6 +135,8 @@ namespace Js
         void EnsureSlots(int oldCount, int newCount, ScriptContext * scriptContext, DynamicTypeHandler * newTypeHandler = nullptr);
         void EnsureSlots(int newCount, ScriptContext *scriptContext);
 
+        DynamicTypeHandler * GetTypeHandler() const;
+
         Var GetSlot(int index);
         Var GetInlineSlot(int index);
         Var GetAuxSlot(int index);

lib/Runtime/Types/DynamicObject.h

@@ -231,8 +231,7 @@ namespace Js
 
         // Check numeric propertyId only if objectArray available
         uint32 indexVal;
-        ScriptContext* scriptContext = instance->GetScriptContext();
-        if (instance->HasObjectArray() && scriptContext->IsNumericPropertyId(propertyId, &indexVal))
+        if (instance->HasObjectArray() && requestContext->IsNumericPropertyId(propertyId, &indexVal))
         {
             return PathTypeHandlerBase::GetItem(instance, originalInstance, indexVal, value, requestContext);
         }

lib/Runtime/Types/PathTypeHandler.cpp

@@ -0,0 +1,39 @@
+//-------------------------------------------------------------------------------------------------------
+// Copyright (C) Microsoft. All rights reserved.
+// Licensed under the MIT license. See LICENSE.txt file in the project root for full license information.
+//-------------------------------------------------------------------------------------------------------
+
+
+var TEST = function(a, b) {
+  if (a != b) {
+    throw new Error(a + " != " + b);
+  }
+}
+
+var obj = { str:6 };
+obj[0] = 'value0'
+obj[6] = 'value6';
+TEST(JSON.stringify(obj, function(k, v) {
+  if (!k) return v;
+  return v + 1
+}), '{"0":"value01","6":"value61","str":7}');
+
+// test ObjectArray
+TEST(JSON.stringify({0:0, 1:1, "two":2}), '{"0":0,"1":1,"two":2}')
+
+var a = new Object();
+
+function replacer(k, v)
+{
+    return v;
+}
+
+var until = (!WScript.Platform || WScript.Platform.BUILD_TYPE == 'Debug') ? 12 : 1290;
+for (var i = 0; i < until; i++)
+{
+    a[i + 10] = 0;
+}
+
+TEST(JSON.stringify(a, replacer).substring(0,20), '{"10":0,"11":0,"12":');
+
+console.log("PASS")

test/JSON/replacerFunction.js

@@ -24,10 +24,7 @@
   </test>
   <test>
     <default>
-      <files>stringify-replacerfunc.js</files>
-      <baseline>stringify-replacerfunc.baseline</baseline>
-      <compile-flags>-recyclerstress</compile-flags>
-      <tags>exclude_fre,Slow</tags>
+      <files>replacerFunction.js</files>
       <timeout>300</timeout>
     </default>
   </test>
@@ -95,4 +92,9 @@
       <baseline>syntaxError.baseline</baseline>
     </default>
   </test>
+  <test>
+    <default>
+      <files>toJSON.js</files>
+    </default>
+  </test>
 </regress-exe>