[PATCH RFA] add dejagnu mips-sim-idt32 baseboard.

[PATCH RFA] add dejagnu mips-sim-idt32 baseboard.

[cgd]

[ MAINTAINERS says to post dejagnu patches here... but doesn't say who
  approves them!  8-]


The following patch adds a mips-sim-idt32 baseboard, which is almost
identical to the existing mips-sim-idt64 baseboard.

The mips-sim-idt64 baseboard is used for mips64 testing, and sets the
ISA markings on the resulting binary to indicate mips64.

This new baseboard is for mips32 testing, and sets the ISA markings to
indicate mips32.

(If you're using a simulator that supports multiple ISAs, e.g. mips32
and mips64, and determines which to support via the binary ISA
markings, then using mips64 isn't The Right Thing: you don't catch
accidental use of mips64 instructions in the code being run.
Therefore, just using e.g. the -idt64 config isn't the right thing for
mips32.)


Once this is approved, i'll "Notify http://dejagnu.sourceforge.net/ of
generic changes." as suggested in the top-level MAINTAINERS file.


chris
[ dejagnu/ChangeLog]
2002-03-11  Chris Demetriou  <cgd@broadcom.com>
 
	* baseboards/mips-sim-idt32.exp: New file.

Index: baseboards/mips-sim-idt32.exp
===================================================================
RCS file: mips-sim-idt32.exp
diff -N mips-sim-idt32.exp
--- /dev/null	Tue May  5 13:32:27 1998
+++ mips-sim-idt32.exp	Mon Mar 11 13:48:44 2002
@@ -0,0 +1,36 @@
+# Load the generic configuration for this board. This will define a basic
+# set of routines needed by the tool to communicate with the board.
+load_generic_config "sim";
+
+# basic-sim.exp is a basic description for the standard Cygnus simulator.
+load_base_board_description "basic-sim";
+
+# This tells it which directory to look in for the simulator.
+setup_sim mips;
+
+# No multilib flags are set by default.
+process_multilib_options "";
+
+# The compiler used to build for this board. This has *nothing* to do
+# with what compiler is tested if we're testing gcc.
+set_board_info compiler "[find_gcc]";
+#set_board_info needs_status_wrapper 1
+
+set_board_info cflags "[libgloss_include_flags] [newlib_include_flags]";
+set_board_info ldflags "[libgloss_link_flags] [newlib_link_flags]";
+
+# Use idt.
+if { [board_info $board obj_format] == "ecoff" } {
+    set_board_info ldscript "-Wl,-Tidtecoff.ld";
+} else {
+    set_board_info ldscript "-Wl,-Tidt32.ld";
+}
+
+# And, it can't do arguments, and doesn't have real signals.
+set_board_info noargs 1;
+set_board_info gdb,nosignals 1;
+
+# Tell gdb to assume no fpu for -msoft-float compilation
+if {[string match "*soft-float*" $current_target_name]} {
+  set_board_info gdb_init_command "set mipsfpu none"
+}

Re: [PATCH RFA] add dejagnu mips-sim-idt32 baseboard.

[Frank Ch. Eigler]

cgd@broadcom.com writes:

> [ MAINTAINERS says to post dejagnu patches here... but doesn't say who
>   approves them!  8-]

That's because local maintainership of src/dejagnu does not solely
belong to any particular group, and thus nor does approval-granting
privilege.  Basically, if you will stand behind it, and it fails to
arouse discontentment, then consider it "approved".

- FChE

Re: [PATCH RFA] add dejagnu mips-sim-idt32 baseboard.

[Andrew Cagney]

> cgd@broadcom.com writes:
> 
> 
>> [ MAINTAINERS says to post dejagnu patches here... but doesn't say who
>> approves them!  8-]

Its a bit vague but the GDB group consider themselves caretakers for the 
dejagnu/ directory in the src/ repository.  Normally the testsuite 
maintainers (in particular Fernando) picks this up, if that fails, me.

As for new baseboards.  Ok by me :-)

> That's because local maintainership of src/dejagnu does not solely
> belong to any particular group, and thus nor does approval-granting
> privilege.  Basically, if you will stand behind it, and it fails to
> arouse discontentment, then consider it "approved".

New one on me!

Andrew

Re: [PATCH RFA] add dejagnu mips-sim-idt32 baseboard.

[cgd]

At Mon, 11 Mar 2002 22:04:26 +0000 (UTC), cgd@broadcom.com wrote:
> [ dejagnu/ChangeLog]
> 2002-03-11  Chris Demetriou  <cgd@broadcom.com>
>  
> 	* baseboards/mips-sim-idt32.exp: New file.

since there was no great hue and cry against this, i've checked it
in.  8-)


chris
-- 
Chris Demetriou                                            Broadcom Corporation
Senior Staff Design Engineer                  Broadband Processor Business Unit
  Any opinions expressed in this message are mine, not necessarily Broadcom's.