* Makefile.in (autoconf-common autoheader-common): Pass `-l

../common' to autoconf and autoheader.  Unconditionally run
	autoconf in every subdir.
	(autoconf-changelog autoheader-changelog): Unconditionally run
	commands in every subdir.
	(autoconf-install autoheader-install): Likewise.

sim/ChangeLog

@@ -1,3 +1,19 @@
+Fri Apr 24 11:14:13 1998  Tom Tromey  <tromey@cygnus.com>
+
+	* Makefile.in (autoconf-common autoheader-common): Pass `-l
+	../common' to autoconf and autoheader.  Unconditionally run
+	autoconf in every subdir.
+	(autoconf-changelog autoheader-changelog): Unconditionally run
+	commands in every subdir.
+	(autoconf-install autoheader-install): Likewise.
+
+Tue Mar 24 17:12:43 1998  Stu Grossman  <grossman@bhuna.cygnus.co.uk>
+
+	* Makefile.in:  Get SHELL from configure.
+	* (FLAGS_TO_PASS):  Pass down SHELL.
+	* configure:  Regenerate with autoconf 2.12.1 to fix shell issues for
+	NT native builds.
+
 Tue Mar 24 11::18:00 1998  Joyce Janczyn  <janczyn@cygnus.com>
 
 	* configure.in (extra_subdirs): Enable igen for mn10300.

sim/Makefile.in

@@ -1,5 +1,5 @@
 #    Makefile template for Configure for the sim library.
-#    Copyright (C) 1993, 1995, 1997 Free Software Foundation, Inc.
+#    Copyright (C) 1993, 1995, 1997, 1998 Free Software Foundation, Inc.
 #    Written by Cygnus Support.
 # 
 # This file is part of BFD, the Binary File Descriptor library.
@@ -45,7 +45,7 @@ man9dir = $(mandir)/man9
 infodir = @infodir@
 includedir = @includedir@
 
-SHELL = /bin/sh
+SHELL = @SHELL@
 
 INSTALL = @INSTALL@
 INSTALL_PROGRAM = @INSTALL_PROGRAM@
@@ -93,7 +93,8 @@ FLAGS_TO_PASS = \
 	"INSTALL_DATA=$(INSTALL_DATA)" \
 	"INSTALL_PROGRAM=$(INSTALL_PROGRAM)" \
 	"RUNTEST=$(RUNTEST)" \
-	"RUNTESTFLAGS=$(RUNTESTFLAGS)"
+	"RUNTESTFLAGS=$(RUNTESTFLAGS)" \
+	"SHELL=$(SHELL)"
 
 # The use of $$(x_FOR_TARGET) reduces the command line length by not
 # duplicating the lengthy definition.
@@ -209,16 +210,13 @@ autoconf-common autoheader-common:
 	for d in * ; \
 	do \
 	    if [ -d $$d -a -f $$d/configure.in ] ; \
-	    then \
-		if grep SIM_AC_COMMON $$d/configure.in >/dev/null ; \
 	    then \
 		 echo "Running autoconf in $$d ..." ; \
-		    (cd $$d && autoconf) ; \
+		 (cd $$d && autoconf -l ../common) ; \
-		    if [ $* = autoheader-common ] ; \
+		 if [ $@ = autoheader-common ] ; \
 		 then \
 		   echo "Running autoheader in $$d ..." ; \
-		      (cd $$d && autoheader) ; \
+		   (cd $$d && autoheader -l ../common) ; \
-		    fi ; \
 		 fi ; \
 	    fi ; \
 	done
@@ -232,14 +230,12 @@ autoconf-changelog autoheader-changelog:
 	for d in * ; \
 	do \
 	    if [ -d $$d -a -f $$d/configure.in ] ; \
-	    then \
-		if grep SIM_AC_COMMON $$d/configure.in >/dev/null ; \
 	    then \
 		 echo "Creating new-ChangeLog in $$d ..." ; \
 		 ( echo "$$date  $$name  <$$id@$$host>" ; \
 		   echo "" ; \
 		   echo "	* configure: Regenerated to track ../common/aclocal.m4 changes." ; \
-		      if [ $* = autoheader-changelog ] ; \
+		   if [ $@ = autoheader-changelog ] ; \
 		   then \
 		     echo "	* config.in: Ditto." ; \
 		   fi ; \
@@ -247,18 +243,14 @@ autoconf-changelog autoheader-changelog:
 		   cat $$d/ChangeLog \
 		 ) > $$d/new-ChangeLog ; \
 	    fi ; \
-	    fi ; \
 	done
 
 autoconf-install autoheader-install:
 	for d in * ; \
 	do \
 	    if [ -d $$d -a -f $$d/configure.in ] ; \
-	    then \
-		if grep SIM_AC_COMMON $$d/configure.in >/dev/null ; \
 	    then \
 		 echo "Moving $$d/new-ChangeLog to $$d/ChangeLog ..." ; \
 		 mv $$d/new-ChangeLog $$d/ChangeLog ; \
 	    fi ; \
-	    fi ; \
 	done

sim/README-HACKING

@@ -8,6 +8,8 @@ Contents
 - TAGS support
 - Generating "configure" files
 - tconfig.in
+- C Language Assumptions
+- "dump" commands under gdb
 
 The "common" directory
 ======================
@@ -37,7 +39,6 @@ The configure.in of a simulator using the common framework should look like:
 
 --- snip ---
 dnl Process this file with autoconf to produce a configure script.
-sinclude(../common/aclocal.m4)
 AC_PREREQ(2.5)dnl
 AC_INIT(Makefile.in)
 
@@ -128,30 +129,30 @@ Generating "configure" files
 ============================
 
 For target's using the common framework, "configure" can be generated
-by running autoconf.  This works because configure.in contains
+by running `autoconf -l ../common'.  This will ensure that the
-"sinclude(../common/aclocal.m4)".
+qaclocal.m4 from the `common' directory is found.
 
 To regenerate the configure files for all targets using the common framework:
 
 	$  cd devo/sim
-	$  make -f Makefile.in autoconf-common
+	$  make -f Makefile.in SHELL=/bin/sh autoconf-common
 
 To add a change-log entry to the ChangeLog file for each updated
 directory (WARNING - check the modified new-ChangeLog files before
 renaming):
 
-	$  make -f Makefile.in autoconf-changelog
+	$  make -f Makefile.in SHELL=/bin/sh autoconf-changelog
 	$  more */new-ChangeLog
-	$  make -f Makefile.in autoconf-install
+	$  make -f Makefile.in SHELL=/bin/sh autoconf-install
 
 In a similar vein, both the configure and config.in files can be
 updated using the sequence:
 
 	$  cd devo/sim
-	$  make -f Makefile.in autoheader-common
+	$  make -f Makefile.in SHELL=/bin/sh autoheader-common
-	$  make -f Makefile.in autoheader-changelog
+	$  make -f Makefile.in SHELL=/bin/sh autoheader-changelog
 	$  more */new-ChangeLog
-	$  make -f Makefile.in autoheader-install
+	$  make -f Makefile.in SHELL=/bin/sh autoheader-install
 
 tconfig.in
 ==========
@@ -179,10 +180,47 @@ ANSI C compiler that supports a 64 bit data type.  Consequently:
 
 	o	The type `unsigned' is valid.
 
-However, the user should not assume:
+However, the user should be aware of the following:
 
-	o	`<number>LL' is acceptable.  Microsoft-C
+	o	GCC's `<number>LL' is NOT acceptable.
-		doesn't reconize it.
+		Microsoft-C doesn't reconize it.
 
-	o	`long long' or `_int64' provides a 64 bit
+	o	MSC's `<number>i64' is NOT acceptable.
-		integer data type.
+		GCC doesn't reconize it.
+
+	o	GCC's `long long' MSC's `_int64' can
+		NOT be used to define 64 bit integer data
+		types.
+
+	o	An empty array (eg int a[0]) is not valid.
+
+When building with GCC it is effectivly a requirement that
+--enable-sim-warnings be specified during configuration.
+
+"dump" commands under gdb
+=========================
+
+gdbinit.in contains the following
+
+define dump
+set sim_debug_dump ()
+end
+
+Simulators that define the sim_debug_dump function can then have their
+internal state pretty printed from gdb.
+
+FIXME: This can obviously be made more elaborate.  As needed it will be.
+
+"dump" commands under gdb
+=========================
+
+gdbinit.in contains the following
+
+define dump
+set sim_debug_dump ()
+end
+
+Simulators that define the sim_debug_dump function can then have their
+internal state pretty printed from gdb.
+
+FIXME: This can obviously be made more elaborate.  As needed it will be.