This is the mail archive of the gdb@sources.redhat.com mailing list for the GDB 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]

[PATCH/RFA] sim autoconf conversion fallout


There still a lot of breakage in the sim tree:

* Several simulators no longer build because the common directory
  isn't configured.  AFAICT all similators need that directory, so I
  propose the attached patch.

* The --enable-sim option is essentially a no-op now, since all
  simulators are built unconditionally if gdb is configured for a
  matching target.  Why has the option been retained while all the
  logic that used it is gone?  Why was the logic removed in the first
  place?

* In the past several simulators were only built when compiling with
  gcc.  That code was ripped out.  Why?

The result of these problems is that gdb cannot be built on many, many
systems anymore.  After the attached patch has been applied, my
immediate concern the ERC32 SPARC V7 simulator.  The simulator is
currently broken (and has been for quite some time I guess and even
before that it only built on i386 and sparc hosts).  As a result GDB
won't build for any sparc*-*-* target.  Is there any interest in
retaining this simulator?  If so, we should at least but some of the
removed logic back.  Otherwise, I propose we remove the ERC32
simulator.

Mark


Index: ChangeLog
from  Mark Kettenis  <kettenis@gnu.org>

	* configure.ac: Always configure the common subdirectory.

Index: configure.ac
===================================================================
RCS file: /cvs/src/src/sim/configure.ac,v
retrieving revision 1.2
diff -u -p -r1.2 configure.ac
--- configure.ac 12 Jan 2005 16:23:11 -0000 1.2
+++ configure.ac 16 Jan 2005 19:25:53 -0000
@@ -33,6 +33,8 @@ yes | no) ;;
 *)	AC_MSG_ERROR(bad value ${enableval} given for --enable-sim option) ;;
 esac])
 
+AC_CONFIG_SUBDIRS(common)
+
 # WHEN ADDING ENTRIES TO THIS MATRIX:
 
 # Make sure that the left side always has two dashes.  Otherwise you
@@ -42,13 +44,11 @@ esac])
 
 if test "${enable_sim}" != no; then
    testsuite=no
-   common=no
    igen=no
    case "${target}" in
        arm*-*-* | thumb*-*-* | strongarm*-*-* | xscale-*-*)
            AC_CONFIG_SUBDIRS(arm)
 	   testsuite=yes
-	   common=yes
 	   ;;
        d10v-*-*)
            AC_CONFIG_SUBDIRS(d10v)
@@ -56,27 +56,22 @@ if test "${enable_sim}" != no; then
        frv-*-*)
            AC_CONFIG_SUBDIRS(frv)
 	   testsuite=yes
-	   common=yes
 	   ;;
        h8300*-*-*)
            AC_CONFIG_SUBDIRS(h8300)
 	   testsuite=yes
-	   common=yes
 	   ;;
        m32r-*-*)
            AC_CONFIG_SUBDIRS(m32r)
 	   testsuite=yes
-	   common=yes
 	   ;;
        m68hc11-*-*|m6811-*-*)
            AC_CONFIG_SUBDIRS(m68hc11)
 	   testsuite=yes
-	   common=yes
 	   ;;
        mcore-*-*)
            AC_CONFIG_SUBDIRS(mcore)
 	   testsuite=yes
-	   common=yes
 	   ;;
        mips*-*-*)
            AC_CONFIG_SUBDIRS(mips)
@@ -90,15 +85,12 @@ if test "${enable_sim}" != no; then
        sh*-*-*)
            AC_CONFIG_SUBDIRS(sh)
 	   testsuite=yes
-	   common=yes
 	   ;;
        powerpc*-*-* )
            AC_CONFIG_SUBDIRS(ppc)
-	   common=yes
 	   ;;
        v850*-*-* )
            AC_CONFIG_SUBDIRS(v850)
-	   common=yes
 	   ;;
        sparc*-*-*)
            AC_CONFIG_SUBDIRS(erc32)
@@ -107,9 +99,6 @@ if test "${enable_sim}" != no; then
    if test "$testsuite" = yes; then
       AC_CONFIG_SUBDIRS(testsuite)
    fi
-   if test "$common" = yes; then
-      AC_CONFIG_SUBDIRS(common)
-   fi
    if test "$igen" = yes; then
       AC_CONFIG_SUBDIRS(igen)
    fi


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