[MERGE #4070 @Cellule] Increase the Wasm function size limit to 76543…

…21 bytes Merge pull request #4070 from Cellule:wasm_func_limit Bumping limit according to discussion on WebAssembly/design#1138 Fixes OS#14291237
chakra-core · Oct 26, 2017 · 191d7ff · 191d7ff
2 parents b8e6804 + bb6ab73
commit 191d7ff
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/WasmReader/WasmLimits.h b/lib/WasmReader/WasmLimits.h
@@ -26,7 +26,7 @@ namespace Wasm {
         static const uint32 MaxMemoryInitialPages = 16384;
         static const uint32 MaxMemoryMaximumPages = 65536;
         static const uint32 MaxModuleSize = 1024 * 1024 * 1024;
-        static const uint32 MaxFunctionSize = 128 * 1024;
+        static const uint32 MaxFunctionSize = 7654321;
     public:
         // Use accessors to easily switch to config flags if needed
         static uint32 GetMaxTypes() { return CONFIG_FLAG(WasmIgnoreLimits) ? UINT32_MAX : MaxTypes; }

diff --git a/test/wasm/limits.js b/test/wasm/limits.js
@@ -21,7 +21,7 @@ const MaxBrTableElems = 1000000;
 const MaxMemoryInitialPages = 16384;
 const MaxMemoryMaximumPages = 65536;
 const MaxModuleSize = 1024 * 1024 * 1024;
-const MaxFunctionSize = 128 * 1024;
+const MaxFunctionSize = 7654321;
 
 /* global assert,testRunner */ // eslint rule
 WScript.LoadScriptFile("../UnitTestFrameWork/UnitTestFrameWork.js");