From 53372ab2791f2bd8a02ef21928893d213e6bfe95 Mon Sep 17 00:00:00 2001 From: Johan Brandhorst-Satzkorn Date: Mon, 30 Jan 2023 21:51:40 -0800 Subject: [PATCH] misc: increase node stack size The default NodeJS V8 stack size is 984K, which is not enough to run the regexp or go/parser tests. This commit increases the stack size to 8192K, which removes the stack size limit error. Fixes #56498 Fixes #57614 Change-Id: I357885d420daf259187403deab25ff587defa0fc Reviewed-on: https://go-review.googlesource.com/c/go/+/463994 Reviewed-by: Dmitri Shuralyov Run-TryBot: Johan Brandhorst-Satzkorn Reviewed-by: Dmitri Shuralyov Reviewed-by: Michael Knyszek Reviewed-by: Julien Fabre Auto-Submit: Johan Brandhorst-Satzkorn TryBot-Result: Gopher Robot --- misc/wasm/go_js_wasm_exec | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/misc/wasm/go_js_wasm_exec b/misc/wasm/go_js_wasm_exec index db4d2cb608..ff592579e0 100755 --- a/misc/wasm/go_js_wasm_exec +++ b/misc/wasm/go_js_wasm_exec @@ -11,4 +11,7 @@ while [ -h "$SOURCE" ]; do done DIR="$( cd -P "$( dirname "$SOURCE" )" && pwd )" -exec node "$DIR/wasm_exec_node.js" "$@" +# Increase the V8 stack size from the default of 984K +# to 8192K to ensure all tests can pass without hitting +# stack size limits. +exec node --stack-size=8192 "$DIR/wasm_exec_node.js" "$@" -- 2.44.0