[binutils-gdb/binutils-2_35-branch] [GOLD] Increase --split-stack-adjust-size

Alan Modra amodra@sourceware.org
Thu Oct 8 12:15:56 GMT 2020


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=5cffc4901e1d72da87aa7ff6d22c152726455831

commit 5cffc4901e1d72da87aa7ff6d22c152726455831
Author: Alan Modra <amodra@gmail.com>
Date:   Thu Oct 8 10:27:43 2020 +1030

    [GOLD] Increase --split-stack-adjust-size
    
    For functions with small (< 256 bytes) stack frames, the current x86
    do_calls_non_split ignores --split-stack-adjust-size and, in
    combination with __morestack_non_split, supplies a non-split-stack
    function with at least 0x100000 (1M) available stack.  On powerpc64, a
    default of 0x4000 is not large enough to reliably work with the golang
    testsuite.  This increase the default size to the defacto x86 value.
    
            * options.h (split_stack_adjust_size): Default to 0x100000.
    
    (cherry picked from commit f9ff65d4dffbaf342dce7a8760059c27683cd962)

Diff:
---
 gold/ChangeLog | 4 ++++
 gold/options.h | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/gold/ChangeLog b/gold/ChangeLog
index 1f2ee06817c..7a886fbcef8 100644
--- a/gold/ChangeLog
+++ b/gold/ChangeLog
@@ -1,3 +1,7 @@
+2020-10-08  Alan Modra  <amodra@gmail.com>
+
+	* options.h (split_stack_adjust_size): Default to 0x100000.
+
 2020-09-26  Alan Modra  <amodra@gmail.com>
 
 	* powerpc.cc (Target_powerpc): Rename power10_stubs_ to
diff --git a/gold/options.h b/gold/options.h
index bc211151dee..51d3614e6b6 100644
--- a/gold/options.h
+++ b/gold/options.h
@@ -1260,7 +1260,7 @@ class General_options
 		 "output sections"),
 	      N_("(PowerPC only) Each output section has its own stubs"));
 
-  DEFINE_uint(split_stack_adjust_size, options::TWO_DASHES, '\0', 0x4000,
+  DEFINE_uint(split_stack_adjust_size, options::TWO_DASHES, '\0', 0x100000,
 	      N_("Stack size when -fsplit-stack function calls non-split"),
 	      N_("SIZE"));


More information about the Binutils-cvs mailing list