[Bug go/64836] New: go.test/test/fixedbugs/issue4348.go FAILs

[Bug go/64836] New: go.test/test/fixedbugs/issue4348.go FAILs

[ro at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64836

            Bug ID: 64836
           Summary: go.test/test/fixedbugs/issue4348.go FAILs
           Product: gcc
           Version: 5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: go
          Assignee: ian at airs dot com
          Reporter: ro at gcc dot gnu.org
                CC: cmang at google dot com
              Host: i386-pc-solaris2.1[01], i686-unknown-linux-gnu,
                    sparc-sun-solaris2.1[01]
            Target: i386-pc-solaris2.1[01], i686-unknown-linux-gnu,
                    sparc-sun-solaris2.1[01]
             Build: i386-pc-solaris2.1[01], i686-unknown-linux-gnu,
                    sparc-sun-solaris2.1[01]

In a 32-bit-default compiler, the 64-bit go.test/test/fixedbugs/issue4348.go
test
FAILs on Solaris/x86, Linux/x86, and Solaris SPARC:

FAIL: go.test/test/fixedbugs/issue4348.go   -O  (test for excess errors)

Excess errors:
/vol/gcc/src/hg/trunk/local/gcc/testsuite/go.test/test/fixedbugs/issue4348.go:20:8:
error: array bound overflows

  Rainer

[Bug go/64836] go.test/test/fixedbugs/issue4348.go FAILs

[ro at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64836

Rainer Orth <ro at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |5.0

[Bug go/64836] go.test/test/fixedbugs/issue4348.go FAILs

[ian at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64836

--- Comment #1 from ian at gcc dot gnu.org <ian at gcc dot gnu.org> ---
Author: ian
Date: Tue Feb  3 03:33:21 2015
New Revision: 220364

URL: https://gcc.gnu.org/viewcvs?rev=220364&root=gcc&view=rev
Log:
    PR go/64836
    PR go/64838

compiler: Use int64_t for backend type size and alignment.

Fixes 32-bit host 64-bit target cross-compilation.

    * go-gcc.cc (Gcc_backend::type_size): Change return type to
    int64_t.
    (Gcc_backend::type_alignment): Likewise.
    (Gcc_backend::type_field_alignment): Likewise.
    (Gcc_backend::type_field_offset): Likewise.
    (Gcc_backend::implicit_variable): Change alignment parameter type
    to int64_t.

Modified:
    trunk/gcc/go/ChangeLog
    trunk/gcc/go/go-gcc.cc
    trunk/gcc/go/gofrontend/backend.h
    trunk/gcc/go/gofrontend/expressions.cc
    trunk/gcc/go/gofrontend/expressions.h
    trunk/gcc/go/gofrontend/gogo.cc
    trunk/gcc/go/gofrontend/gogo.h
    trunk/gcc/go/gofrontend/types.cc
    trunk/gcc/go/gofrontend/types.h

[Bug go/64836] go.test/test/fixedbugs/issue4348.go FAILs

[ian at airs dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64836

Ian Lance Taylor <ian at airs dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED

--- Comment #2 from Ian Lance Taylor <ian at airs dot com> ---
Should be fixed.