[Bug go/46959] New: M68K Not Supported by Go

[Bug go/46959] New: M68K Not Supported by Go

[joel at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46959

           Summary: M68K Not Supported by Go
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: go
        AssignedTo: ian@airs.com
        ReportedBy: joel@gcc.gnu.org


m68k-rtems4.11-gcc (GCC) 4.6.0 20101207 (experimental) [trunk revision 167563]

/users/joel/test-gcc/gcc-svn/libgo/runtime/go-go.c:40:3: error: #error must
define SAVE_REGS
/users/joel/test-gcc/gcc-svn/libgo/runtime/go-go.c: In function
'__go_run_goroutine_gc':
/users/joel/test-gcc/gcc-svn/libgo/runtime/go-go.c:348:3: error: 'SAVE_REGS'
undeclared (first use in this function)
/users/joel/test-gcc/gcc-svn/libgo/runtime/go-go.c:348:3: note: each undeclared
identifier is reported only once for each function it appears in
/users/joel/test-gcc/gcc-svn/libgo/runtime/go-go.c: In function
'__go_scanstacks':
/users/joel/test-gcc/gcc-svn/libgo/runtime/go-go.c:437:3: error: 'SAVE_REGS'
undeclared (first use in this function)
make[4]: *** [go-go.lo] Error 1

[Bug go/46959] M68K Not Supported by Go

[joel at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46959

--- Comment #1 from Joel Sherrill <joel at gcc dot gnu.org> 2010-12-15 16:03:49 UTC ---
How does this look for m68k?  I recall d0/d1/a0/a1 are clobbered by the caller.
 a6 is a frame pointer, a7 is the stack pointer.  So is this the set that needs
to be handled for m68k?

#elif defined(__m68k__)
 #define SAVE_REGS asm ("" : : : "a2", "a3", "a4", "a5", "a6", "a7", \
                                 "d2", "d3", "d4", "d5", "d6", "d7" )

[Bug go/46959] M68K Not Supported by Go

[schwab@linux-m68k.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46959

--- Comment #2 from Andreas Schwab <schwab@linux-m68k.org> 2010-12-15 16:40:08 UTC ---
You cannot put reserved registers in the clobber list.

[Bug go/46959] M68K Not Supported by Go

[ian at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46959

--- Comment #3 from ian at gcc dot gnu.org <ian at gcc dot gnu.org> 2011-01-05 00:24:30 UTC ---
Author: ian
Date: Wed Jan  5 00:24:15 2011
New Revision: 168500

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=168500
Log:
    PR go/46959
    PR go/46960
    PR go/46961
    PR go/46962
    PR go/46963
    PR go/46964
Use __builtin_unwind_init rather than the SAVE_REGS hack.

Modified:
    trunk/libgo/runtime/go-go.c

[Bug go/46959] M68K Not Supported by Go

[ian at airs dot com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46959

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

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

--- Comment #4 from Ian Lance Taylor <ian at airs dot com> 2011-01-05 00:36:58 UTC ---
Fixed.