MIPS_STABS_ELF is now broken

[Daniel Jacobowitz]

On Tue, Jun 11, 2002 at 08:45:55AM -0700, H . J . Lu wrote:
> 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.

That's a bogus argument.  Remember, GCC 2.96 had to be changed for
MIPS_STABS_ELF.  We didn't switch to that until around GCC 3.0.3
timeframe.

You changed it last time; change it again.  I think this is perfectly
OK.  Pick up the tiny patch to support DWARF-2 while you're at it.

-- 
Daniel Jacobowitz                           Carnegie Mellon University
MontaVista Software                         Debian GNU/Linux Developer