On Thu, Jul 30, 2015 at 10:46 AM, Lynn A. Boger
wrote:
> Can this patch be submitted to fix the ppc 32 bootstrap error?
>
> On 06/19/2015 07:58 AM, Andreas Schwab wrote:
>>>
>>> * go-lang.c (go_langhook_init_options_struct): Don't set
>>> x_flag_split_stack.
>>> (go_langhoo
Can this patch be submitted to fix the ppc 32 bootstrap error?
On 06/19/2015 07:58 AM, Andreas Schwab wrote:
* go-lang.c (go_langhook_init_options_struct): Don't set
x_flag_split_stack.
(go_langhook_post_options): Set it here instead.
---
gcc/go/go-lang.c | 9 +
I have tested this and it fixes the problem.
On 07/01/2015 09:15 AM, Andreas Schwab wrote:
"Lynn A. Boger" writes:
It's not clear to me what distro/gcc/glibc versions and type of build
causes the error. I have not been able to reproduce the
original problem.
The failure mode is quite obviou
"Lynn A. Boger" writes:
> It's not clear to me what distro/gcc/glibc versions and type of build
> causes the error. I have not been able to reproduce the
> original problem.
The failure mode is quite obvious: go_langhook_init_options_struct is
called before the options are parsed, so -m32 hasn'
If further testing is needed on this patch I can do it, but I need more
information what variations need to be tested?
It's not clear to me what distro/gcc/glibc versions and type of build
causes the error. I have not been able to reproduce the
original problem.
On 06/15/2015 01:58 PM, Andre
> * go-lang.c (go_langhook_init_options_struct): Don't set
> x_flag_split_stack.
> (go_langhook_post_options): Set it here instead.
> ---
> gcc/go/go-lang.c | 9 +
> 1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/gcc/go/go-lang.c b/gcc/go/go-lang.c
> ind
* go-lang.c (go_langhook_init_options_struct): Don't set
x_flag_split_stack.
(go_langhook_post_options): Set it here instead.
---
gcc/go/go-lang.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/gcc/go/go-lang.c b/gcc/go/go-lang.c
index ce4dd9b.
The bug is of course that like DEFAULT_ABI, rs6000_isa_flags hasn't been
determined yet.
Andreas.
--
Andreas Schwab, sch...@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5
"And now for something completely different."
Alan Modra writes:
> This untested patch ought to fix the problem, I think.
There is no -fsplit-stack in the Makefile, and the configure script
has already determined the correct settings.
$ grep -e -fsplit-stack libgo/Makefile 32/libgo/Makefile
libgo/Makefile:SPLIT_STACK = -fsplit-stack
$ gre
Alan Modra writes:
> diff --git a/libgo/configure.ac b/libgo/configure.ac
> index 7c403a5..2ddcdfd 100644
> --- a/libgo/configure.ac
> +++ b/libgo/configure.ac
> @@ -366,6 +366,13 @@ esac
> AC_SUBST(OSCFLAGS)
>
> dnl Use -fsplit-stack when compiling C code if available.
> +case "$target" in
>
On Sat, Jun 13, 2015 at 12:46:18PM +0200, Andreas Schwab wrote:
> /usr/bin/mkdir -p .; files=`echo ../../../../libgo/go/errors/errors.go | sed
> -e 's/[^ ]*\.gox//g'`; /bin/sh ./libtool --tag GO --mode=compile
> /daten/gcc/gcc-20150613/Build/./gcc/gccgo
> -B/daten/gcc/gcc-20150613/Build/./gcc/ -
/usr/bin/mkdir -p .; files=`echo ../../../../libgo/go/errors/errors.go | sed -e
's/[^ ]*\.gox//g'`; /bin/sh ./libtool --tag GO --mode=compile
/daten/gcc/gcc-20150613/Build/./gcc/gccgo
-B/daten/gcc/gcc-20150613/Build/./gcc/ -B/usr/powerpc64-linux/bin/
-B/usr/powerpc64-linux/lib/ -isystem /usr/po
The following series of patches add -fsplit-stack support for
powerpc64-linux. Each was cumulatively bootstrapped and regression
tested powerpc64-linux and powerpc64le-linux.
--
Alan Modra
Australia Development Lab, IBM
13 matches
Mail list logo
