[Bug go/83787] [8 regression] Many 32-bit Solaris/SPARC Go tests FAIL after Go1.10beta1 update

2018-01-17 Thread ian at airs dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83787

Ian Lance Taylor  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

--- Comment #2 from Ian Lance Taylor  ---
Should be fixed.

[Bug go/83787] [8 regression] Many 32-bit Solaris/SPARC Go tests FAIL after Go1.10beta1 update

2018-01-17 Thread ian at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83787

--- Comment #1 from ian at gcc dot gnu.org  ---
Author: ian
Date: Thu Jan 18 04:24:48 2018
New Revision: 256835

URL: https://gcc.gnu.org/viewcvs?rev=256835=gcc=rev
Log:
PR go/83787
compiler: pass int to makechan, call makechan64 when appropriate

The update to 1.10beta1 changed makechan to take int instead of int64,
and added a makechan64 call for large values.  Since the size is the
last argument to makechan, the old compiler which always passed a
64-bit int worked fine on 64-bit systems and little-endian 32-bit
systems, but broke on big-endian 32-bit systems.  This CL fixes the
compiler to use the appropriate types.

This fixes GCC PR 83787.

Reviewed-on: https://go-review.googlesource.com/88077

Modified:
trunk/gcc/go/gofrontend/MERGE
trunk/gcc/go/gofrontend/escape.cc
trunk/gcc/go/gofrontend/expressions.cc
trunk/gcc/go/gofrontend/runtime.def
trunk/libgo/go/runtime/chan.go

[Bug go/83787] [8 regression] Many 32-bit Solaris/SPARC Go tests FAIL after Go1.10beta1 update

2018-01-11 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83787

Jakub Jelinek  changed:

   What|Removed |Added

   Priority|P3  |P4
 CC||jakub at gcc dot gnu.org

[Bug go/83787] [8 regression] Many 32-bit Solaris/SPARC Go tests FAIL after Go1.10beta1 update

2018-01-11 Thread ro at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83787

Rainer Orth  changed:

   What|Removed |Added

   Target Milestone|--- |8.0