Add -g3 on some platforms even if .devel isn't present.

Now that we're requiring autoconf 2.61 or later, AC_PROG_CC sets -g
along with -O2 for GCC-like compilers regardless of whether .devel is
present.  Add "include debugging symbols" options for MIPS C/DEC C
regardless of whether .devel is present; that's -g3, which is the
version that doesn't turn optimization off (we also turn on -O).

Also, don't change the ABI on IRIX depending on .devel.

I'm not sure why we were doing that; if somebody has a good reason to
continue doing it, please let us know what it is.

Expand some comments while we're at it.

diff --git a/aclocal.m4 b/aclocal.m4
index 54f0757d91b25af83b235380411ae0f30d8767d9..59ddcfcab76396bbf1c94f13d64ed8f415050895 100644 (file)
--- a/aclocal.m4
+++ b/aclocal.m4
@@ -76,6 +76,7 @@ dnl If using cc:
 dnl     require that it support ansi prototypes
 dnl     use -O (AC_PROG_CC will use -g -O2 on gcc, so we don't need to
 dnl     do that ourselves for gcc)
+dnl     add -g flags, as appropriate
 dnl     explicitly specify /usr/local/include
 dnl
 dnl NOTE WELL: with newer versions of autoconf, "gcc" means any compiler
@@ -160,7 +161,13 @@ AC_DEFUN(AC_LBL_C_INIT,
                    # 1989 and the early '90's, so maybe we can just
                    # drop support for those compilers.)
                    #
-                   $1="$$1 -xansi -signed"
+                   # -g is equivalent to -g2, which turns off
+                   # optimization; we choose -g3, which generates
+                   # debugging information but doesn't turn off
+                   # optimization (even if the optimization would
+                   # cause inaccuracies in debugging).
+                   #
+                   $1="$$1 -xansi -signed -g3"
                    ;;
 
            osf*)
@@ -175,6 +182,14 @@ AC_DEFUN(AC_LBL_C_INIT,
                    # don't want to try using GCC-style -W flags.
                    #
                    ac_lbl_cc_dont_try_gcc_dashW=yes
+                   #
+                   # -g is equivalent to -g2, which turns off
+                   # optimization; we choose -g3, which generates
+                   # debugging information but doesn't turn off
+                   # optimization (even if the optimization would
+                   # cause inaccuracies in debugging).
+                   #
+                   $1="$$1 -g3"
                    ;;
 
            solaris*)
@@ -941,7 +956,6 @@ dnl
 dnl If the file .devel exists:
 dnl    Add some warning flags if the compiler supports them
 dnl    If an os prototype include exists, symlink os-proto.h to it
-dnl     Add -g flags, as appropriate, for various non-gcc-style compilers
 dnl
 dnl usage:
 dnl
@@ -971,28 +985,14 @@ AC_DEFUN(AC_LBL_DEVEL,
                    AC_LBL_CHECK_COMPILER_OPT($1, -W)
            fi
            AC_LBL_CHECK_DEPENDENCY_GENERATION_OPT()
-           if test "$GCC" != yes ; then
-                   case "$host_os" in
-
-                   irix6*)
-                           # Presumed to be MIPS C.
-                           V_CCOPT="$V_CCOPT -n32 -g3"
-                           ;;
-
-                   irix*)
-                           # Presumed to be MIPS C.
-                           V_CCOPT="$V_CCOPT -g3"
-                           ;;
-
-                   osf*)
-                           # Presumed to be the DEC C compiler.
-                           V_CCOPT="$V_CCOPT -g3"
-                           ;;
-
-                   *)
-                           ;;
-                   esac
-           fi
+           #
+           # We used to set -n32 for IRIX 6 when not using GCC (presumed
+           # to mean that we're using MIPS C or MIPSpro C); it specified
+           # the "new" faster 32-bit ABI, introduced in IRIX 6.2.  I'm
+           # not sure why that would be something to do *only* with a
+           # .devel file; why should the ABI for which we produce code
+           # depend on .devel?
+           #
            os=`echo $host_os | sed -e 's/\([[0-9]][[0-9]]*\)[[^0-9]].*$/\1/'`
            name="lbl/os-$os.h"
            if test -f $name ; then

diff --git a/configure b/configure
index a387079727cf7679188778ddc7303b3ce25af666..1356816caf96433388b0e5b775914a07c892d5e2 100755 (executable)
--- a/configure
+++ b/configure
@@ -3421,7 +3421,13 @@ rm -f core conftest.err conftest.$ac_objext conftest.$ac_ext
                    # 1989 and the early '90's, so maybe we can just
                    # drop support for those compilers.)
                    #
-                   V_CCOPT="$V_CCOPT -xansi -signed"
+                   # -g is equivalent to -g2, which turns off
+                   # optimization; we choose -g3, which generates
+                   # debugging information but doesn't turn off
+                   # optimization (even if the optimization would
+                   # cause inaccuracies in debugging).
+                   #
+                   V_CCOPT="$V_CCOPT -xansi -signed -g3"
                    ;;
 
            osf*)
@@ -3436,6 +3442,14 @@ rm -f core conftest.err conftest.$ac_objext conftest.$ac_ext
                    # don't want to try using GCC-style -W flags.
                    #
                    ac_lbl_cc_dont_try_gcc_dashW=yes
+                   #
+                   # -g is equivalent to -g2, which turns off
+                   # optimization; we choose -g3, which generates
+                   # debugging information but doesn't turn off
+                   # optimization (even if the optimization would
+                   # cause inaccuracies in debugging).
+                   #
+                   V_CCOPT="$V_CCOPT -g3"
                    ;;
 
            solaris*)
@@ -7756,28 +7770,14 @@ $as_echo "no" >&6; }
 
 
 
-           if test "$GCC" != yes ; then
-                   case "$host_os" in
-
-                   irix6*)
-                           # Presumed to be MIPS C.
-                           V_CCOPT="$V_CCOPT -n32 -g3"
-                           ;;
-
-                   irix*)
-                           # Presumed to be MIPS C.
-                           V_CCOPT="$V_CCOPT -g3"
-                           ;;
-
-                   osf*)
-                           # Presumed to be the DEC C compiler.
-                           V_CCOPT="$V_CCOPT -g3"
-                           ;;
-
-                   *)
-                           ;;
-                   esac
-           fi
+           #
+           # We used to set -n32 for IRIX 6 when not using GCC (presumed
+           # to mean that we're using MIPS C or MIPSpro C); it specified
+           # the "new" faster 32-bit ABI, introduced in IRIX 6.2.  I'm
+           # not sure why that would be something to do *only* with a
+           # .devel file; why should the ABI for which we produce code
+           # depend on .devel?
+           #
            os=`echo $host_os | sed -e 's/\([0-9][0-9]*\)[^0-9].*$/\1/'`
            name="lbl/os-$os.h"
            if test -f $name ; then