This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: MIPS_STABS_ELF is now broken


On Tue, Jun 11, 2002 at 10:01:08AM -0400, Daniel Jacobowitz wrote:
> On Mon, Jun 10, 2002 at 11:38:53PM -0700, H . J . Lu wrote:
> > Hi Daniel,
> > 
> > I believe your patch
> > 
> > http://sources.redhat.com/ml/binutils/2002-06/msg00194.html
> > 
> > breaks MIPS_STABS_ELF.
> > 
> > # gcc -c x.c
> > # objdump --full -j .stabstr  x.o|m 
> > 
> > x.o:     file format elf32-tradlittlemips
> > 
> > Contents of section .stabstr:
> >  0000 002f746d 702f6363 4b305a32 6f452e73  ./tmp/ccK0Z2oE.s
> > 					   ^^^^^^^^^^^^^^^^
> > 
> > It was not there before your change. As the result, gcc bootstrap fails
> > in "make compare" when MIPS_STABS_ELF is used. Could you please fix it?
> > I don't think you should pass the first .file to dwarf. See
> > 
> > http://sources.redhat.com/ml/binutils/2002-06/msg00066.html
> > 
> > and how alpha handles it.
> > 
> > Thanks.
> 
> As Eric said, this should be fixed in GCC; that .file is completely
> bogus.  If it contains the path in /tmp, that's even more true.
> 

How does it work with the existing gcc binary? I don't believe gcc should
be changed at all. You can't require the new gcc for the old working
feature. It is ok to require gcc 3.1.1 for dwarf. But it is not ok to
require a new gcc 2.96 because you changed the MIPS_STABS_ELF in gas.
Please make sure gas is compatible with all versions of gcc which support
MIPS_STABS_ELF.


H.J.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]