Re: 3.3 PATCH: Handle mips gas 2.13 -mdebug switch

[Eric Christopher <echristo at redhat dot com>]

> Indeed: unfortunately, I'm currently hampered by a bug (yet to be found) in
> gas 2.12.90 which causes many additional testsuite failures for my irix6o32
> configuration with stabs, compared to 2.12.1.  So I need to support the
> latter for some time.  Testresults for gas 2.12.1 with stabs are idential
> to gas 2.12.90 with DWARF-2 :-).
> 

Hrm. If you can show what it is I'll help you track it down.

> > 1) I wish there was a way to require certain versions of gas with
> > certain versions of gcc.
> 
> This is currently done both by feature testing and mentioning the
> requirements in install.texi (which too few people read, unfortunately).
> 

Yeah, I'd really like to require it, but...

> > 3) I'm trying to get rid of SDB_DEBUGGING, in a patch I'll be committing
> > shortly I'm moving everything but mips-ecoff, mips-sni, and irix5. I'm
> > not sure about the o32 irix6.
> 
> I cannot really parse this: irix6o32 only inherits SDB_DEBUGGING from
> iris5gas.h, so if you remove it there, irix6o32 will loose it as well.  I
> certainly agree that this is useful: gdb support is probably even more
> bit-rotten than stabs-in-mdebug.
> 

Hrm. OK, well, I'll happily accept a patch to remove it from that. :)

> > 4) The patch in general looks ok to me after these comments. You'll need
> > more approval than me though.
> 
> Ok, I fear I'll need a global-write-privs maintainer for the gcc.c (and
> configure.in) parts.  Any takers?  The configure.in part is below, the rest
> of the patch at
> 
> 	http://gcc.gnu.org/ml/gcc-patches/2002-07/msg01070.html
> 

You will need someone else to approve, I don't see a problem with it
though.

-eric

-- 
I will not grease the monkey bars