Re: other/8314: m68k-elf/rtems should multilib crt[begin|end].o

Re: other/8314: m68k-elf/rtems should multilib crt[begin|end].o

[Joel Sherrill]

The following reply was made to PR other/8314; it has been noted by GNATS.

From: Joel Sherrill <joel.sherrill@OARcorp.com>
To: mmitchel@gcc.gnu.org, ccj@acm.org, gcc-bugs@gcc.gnu.org,
 	gcc-prs@gcc.gnu.org, joel@gcc.gnu.org, law@gcc.gnu.org,
 	pirovano.thysys@tiscalinet.it, gcc-gnats@gcc.gnu.org
Cc:  
Subject: Re: other/8314: m68k-elf/rtems should multilib crt[begin|end].o
Date: Tue, 12 Nov 2002 08:14:24 -0600

 This also applies to the head.  Was it applied there as well?
 
 mmitchel@gcc.gnu.org wrote:
 > 
 > Synopsis: m68k-elf/rtems should multilib crt[begin|end].o
 > 
 > State-Changed-From-To: feedback->closed
 > State-Changed-By: mmitchel
 > State-Changed-When: Sun Nov 10 22:03:45 2002
 > State-Changed-Why:
 >     Fixed in GCC 3.2.1 by applying the patch in the audit trail.
 > 
 > http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=8314
 
 -- 
 Joel Sherrill, Ph.D.             Director of Research & Development
 joel@OARcorp.com                 On-Line Applications Research
 Ask me about RTEMS: a free RTOS  Huntsville AL 35805
 Support Available                (256) 722-9985

Re: other/8314: m68k-elf/rtems should multilib crt[begin|end].o

[mmitchel]

Synopsis: m68k-elf/rtems should multilib crt[begin|end].o

State-Changed-From-To: feedback->closed
State-Changed-By: mmitchel
State-Changed-When: Sun Nov 10 22:03:45 2002
State-Changed-Why:
    Fixed in GCC 3.2.1 by applying the patch in the audit trail.

http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=8314

Re: other/8314: m68k-elf/rtems should multilib crt[begin|end].o

[joel]

Synopsis: m68k-elf/rtems should multilib crt[begin|end].o

Responsible-Changed-From-To: unassigned->law
Responsible-Changed-By: joel
Responsible-Changed-When: Wed Oct 23 10:15:25 2002
Responsible-Changed-Why:
    Jeff is listed as m68k maintainer.

http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=8314

Re: other/8314: m68k-elf/rtems should multilib crt[begin|end].o

[joel]

Synopsis: m68k-elf/rtems should multilib crt[begin|end].o

State-Changed-From-To: open->feedback
State-Changed-By: joel
State-Changed-When: Tue Oct 22 15:50:25 2002
State-Changed-Why:
    Original patch broken.  Testing this now:
    
    diff -uNr /usr1/rtems/work-tools/original/gcc-3.2/gcc/config/m68k/t-crtstuff gcc-3.2/gcc/config/m68k/t-crtstuff
    --- /usr1/rtems/work-tools/original/gcc-3.2/gcc/config/m68k/t-crtstuff  Tue Sep 14 04:52:09 1999
    +++ gcc-3.2/gcc/config/m68k/t-crtstuff  Tue Oct 22 17:47:14 2002
    @@ -1,11 +1,10 @@
    -# from ../t-svr4
    -EXTRA_PARTS=crtbegin.o crtend.o crti.o crtn.o
    +EXTRA_MULTILIB_PARTS=crtbegin.o crtend.o crti.o crtn.o
    
     # Add flags here as required.
     CRTSTUFF_T_CFLAGS =
    
     # Assemble startup files.
    -crti.o: $(srcdir)/config/m68k/crti.s $(GCC_PASSES)
    -       $(GCC_FOR_TARGET) -c -o crti.o $(srcdir)/config/m68k/crti.s
    -crtn.o: $(srcdir)/config/m68k/crtn.s $(GCC_PASSES)
    -       $(GCC_FOR_TARGET) -c -o crtn.o $(srcdir)/config/m68k/crtn.s
    +$(T)crti.o: $(srcdir)/config/m68k/crti.s $(GCC_PASSES)
    +       $(GCC_FOR_TARGET) -c -o $(T)crti.o $(srcdir)/config/m68k/crti.s
    +$(T)crtn.o: $(srcdir)/config/m68k/crtn.s $(GCC_PASSES)
    +       $(GCC_FOR_TARGET) -c -o $(T)crtn.o $(srcdir)/config/m68k/crtn.s

http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=8314

other/8314: m68k-elf/rtems should multilib crt[begin|end].o

[joel]

>Number:         8314
>Category:       other
>Synopsis:       m68k-elf/rtems should multilib crt[begin|end].o
>Confidential:   no
>Severity:       critical
>Priority:       medium
>Responsible:    unassigned
>State:          open
>Class:          wrong-code
>Submitter-Id:   net
>Arrival-Date:   Tue Oct 22 15:46:00 PDT 2002
>Closed-Date:
>Last-Modified:
>Originator:     joel@gcc.gnu.org
>Release:        gcc 3.2
>Organization:
>Environment:
Target m68k-elf and m68k-rtems
>Description:
crtbegin.o and crtend.o are compiled from C and 
when must be compiled with appropriate CPU options
to generate correct code on coldfire models. 
>How-To-Repeat:

>Fix:
2002-10-22	Joel Sherrill <joel@OARcorp.com>

	* config/m68k/t-crtstuff: Include crti.o, crtn.o, crtbegin.o
	and crtend.o as multilib items.

diff -uNr /usr1/rtems/work-tools/original/gcc-3.2/gcc/config/m68k/t-crtstuff gcc-3.2/gcc/config/m68k/t-crtstuff
--- /usr1/rtems/work-tools/original/gcc-3.2/gcc/config/m68k/t-crtstuff  Tue Sep 14 04:52:09 1999
+++ gcc-3.2/gcc/config/m68k/t-crtstuff  Tue Oct 22 17:24:52 2002
@@ -1,5 +1,4 @@
-# from ../t-svr4
-EXTRA_PARTS=crtbegin.o crtend.o crti.o crtn.o
+EXTRA_MULTILIB_PARTS=crtbegin.o crtend.o crti.o crtn.o

 # Add flags here as required.
 CRTSTUFF_T_CFLAGS =
>Release-Note:
>Audit-Trail:
>Unformatted: