This is the mail archive of the
binutils@sources.redhat.com
mailing list for the binutils project.
Re: 3.3 PATCH: Handle mips gas 2.13 -mdebug switch
Eric Christopher writes:
> 0) I was just working on this myself. If you require #1 it's much easier
> ;)
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 :-).
> 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).
> 2) The configure part of this patch is missing :)
Right: included below. This part was mostly cut-and-paste, but anyway
tested on the various assembler versions indicated in the patch submission.
> 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.
> 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
Thanks.
Rainer
Index: gcc/configure.in
===================================================================
RCS file: /cvs/gcc/gcc/gcc/configure.in,v
retrieving revision 1.610
diff -u -p -r1.610 configure.in
--- gcc/configure.in 14 Jul 2002 01:59:13 -0000 1.610
+++ gcc/configure.in 19 Jul 2002 17:03:38 -0000
@@ -2154,6 +2156,35 @@ if test x"$gcc_cv_as_gstabs_flag" = xyes
fi
AC_MSG_RESULT($gcc_cv_as_gstabs_flag)
+case "$target" in
+ mips*-*-*)
+ AC_MSG_CHECKING(assembler -mdebug support)
+ gcc_cv_as_mdebug_flag=no
+ if test x$gcc_cv_gas_major_version != x -a x$gcc_cv_gas_minor_version != x;
+ then
+ if test "$gcc_cv_gas_major_version" -eq 2 \
+ -a "$gcc_cv_gas_minor_version" -ge 13 \
+ -o "$gcc_cv_gas_major_version" -gt 2 \
+ && grep 'obj_format = elf' ../gas/Makefile > /dev/null \
+ && test x"$insn" != x ; then
+ gcc_cv_as_mdebug_flag="yes"
+ fi
+ elif test x$gcc_cv_as != x -a x"$insn" != x ; then
+ echo '' > conftest.s
+ # ??? This fails with non-gnu grep.
+ if $gcc_cv_as -mdebug -o conftest.o conftest.s > /dev/null 2>&1 ; then
+ gcc_cv_as_mdebug_flag="yes"
+ fi
+ rm -f conftest.s conftest.o
+ fi
+ if test x"$gcc_cv_as_mdebug_flag" = xyes; then
+ AC_DEFINE(HAVE_AS_MDEBUG_DEBUG_FLAG, 1,
+ [Define if your assembler supports the -mdebug option.])
+ fi
+ AC_MSG_RESULT($gcc_cv_as_mdebug_flag)
+ ;;
+esac
+
AC_MSG_CHECKING(linker PT_GNU_EH_FRAME support)
gcc_cv_ld_eh_frame_hdr=no
if test x$gcc_cv_gld_major_version != x -a x$gcc_cv_gld_minor_version != x; then