Assume TARGET_BYTE_ORDER_SELECTABLE{,_P} is always true.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 1b4ef88494076f84f6267177d99bb86733419f59..68a6090a959c79469d17924be35aa46abf79b2d3 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,28 @@
+2002-01-20  Andrew Cagney  <ac131313@redhat.com>
+
+       * remote-sim.c (gdbsim_open): Simplify code testing the macro
+       TARGET_BYTE_ORDER_SELECTABLE_P.  Assume the target is always
+       byte-order selectable.
+       * sparc-tdep.c (sparc_target_architecture_hook): Ditto.
+       * arch-utils.c: Ditto.
+       (set_endian): Ditto.
+       (set_endian_from_file): Ditto.
+       * gdbserver/low-sim.c (create_inferior): Ditto.
+       * gdbarch.sh: Ditto.
+       * gdbarch.h: Re-generate.
+       * config/powerpc/tm-ppc-eabi.h (TARGET_BYTE_ORDER_SELECTABLE_P): 
+       * config/sparc/tm-sparclite.h (TARGET_BYTE_ORDER_SELECTABLE): 
+       * config/sparc/tm-sparclet.h (TARGET_BYTE_ORDER_SELECTABLE): 
+       * config/mcore/tm-mcore.h (TARGET_BYTE_ORDER_SELECTABLE_P): 
+       * config/arm/tm-wince.h (TARGET_BYTE_ORDER_SELECTABLE_P): 
+       * config/arm/tm-linux.h (TARGET_BYTE_ORDER_SELECTABLE_P): 
+       * config/arc/tm-arc.h (TARGET_BYTE_ORDER_SELECTABLE): 
+       * config/arm/tm-arm.h (TARGET_BYTE_ORDER_SELECTABLE_P): Delete
+       macro definition.
+       * config/mips/tm-wince.h: Remove #undef of macro
+       TARGET_BYTE_ORDER_SELECTABLE.
+       * config/sh/tm-wince.h: Ditto.
+
 2002-01-20  Daniel Jacobowitz  <drow@mvista.com>
 
        * gdbtypes.h (struct cplus_struct_type): Add is_artificial to

diff --git a/gdb/arch-utils.c b/gdb/arch-utils.c
index b99e62a9befbdd0eeea2b19e18e447cd149cd668..ea95cdea3849fa52c83cf5e9b8fd22afdab6ab8b 100644 (file)
--- a/gdb/arch-utils.c
+++ b/gdb/arch-utils.c
@@ -399,23 +399,6 @@ generic_register_virtual_size (int regnum)
 \f
 /* Functions to manipulate the endianness of the target.  */
 
-#ifdef TARGET_BYTE_ORDER_SELECTABLE
-/* compat - Catch old targets that expect a selectable byte-order to
-   default to BFD_ENDIAN_BIG */
-#ifndef TARGET_BYTE_ORDER_DEFAULT
-#define TARGET_BYTE_ORDER_DEFAULT BFD_ENDIAN_BIG
-#endif
-#endif
-#if !TARGET_BYTE_ORDER_SELECTABLE_P
-#ifndef TARGET_BYTE_ORDER_DEFAULT
-/* compat - Catch old non byte-order selectable targets that do not
-   define TARGET_BYTE_ORDER_DEFAULT and instead expect
-   TARGET_BYTE_ORDER to be used as the default.  For targets that
-   defined neither TARGET_BYTE_ORDER nor TARGET_BYTE_ORDER_DEFAULT the
-   below will get a strange compiler warning. */
-#define TARGET_BYTE_ORDER_DEFAULT TARGET_BYTE_ORDER
-#endif
-#endif
 #ifndef TARGET_BYTE_ORDER_DEFAULT
 #define TARGET_BYTE_ORDER_DEFAULT BFD_ENDIAN_BIG /* arbitrary */
 #endif
@@ -453,11 +436,7 @@ show_endian (char *args, int from_tty)
 static void
 set_endian (char *ignore_args, int from_tty, struct cmd_list_element *c)
 {
-  if (!TARGET_BYTE_ORDER_SELECTABLE_P)
-    {
-      printf_unfiltered ("Byte order is not selectable.");
-    }
-  else if (set_endian_string == endian_auto)
+  if (set_endian_string == endian_auto)
     {
       target_byte_order_auto = 1;
     }
@@ -508,33 +487,20 @@ set_endian (char *ignore_args, int from_tty, struct cmd_list_element *c)
 static void
 set_endian_from_file (bfd *abfd)
 {
+  int want;
   if (GDB_MULTI_ARCH)
     internal_error (__FILE__, __LINE__,
                    "set_endian_from_file: not for multi-arch");
-  if (TARGET_BYTE_ORDER_SELECTABLE_P)
-    {
-      int want;
-      
-      if (bfd_big_endian (abfd))
-       want = BFD_ENDIAN_BIG;
-      else
-       want = BFD_ENDIAN_LITTLE;
-      if (TARGET_BYTE_ORDER_AUTO)
-       target_byte_order = want;
-      else if (TARGET_BYTE_ORDER != want)
-       warning ("%s endian file does not match %s endian target.",
-                want == BFD_ENDIAN_BIG ? "big" : "little",
-                TARGET_BYTE_ORDER == BFD_ENDIAN_BIG ? "big" : "little");
-    }
+  if (bfd_big_endian (abfd))
+    want = BFD_ENDIAN_BIG;
   else
-    {
-      if (bfd_big_endian (abfd)
-         ? TARGET_BYTE_ORDER != BFD_ENDIAN_BIG
-         : TARGET_BYTE_ORDER == BFD_ENDIAN_BIG)
-       warning ("%s endian file does not match %s endian target.",
-                bfd_big_endian (abfd) ? "big" : "little",
-                TARGET_BYTE_ORDER == BFD_ENDIAN_BIG ? "big" : "little");
-    }
+    want = BFD_ENDIAN_LITTLE;
+  if (TARGET_BYTE_ORDER_AUTO)
+    target_byte_order = want;
+  else if (TARGET_BYTE_ORDER != want)
+    warning ("%s endian file does not match %s endian target.",
+            want == BFD_ENDIAN_BIG ? "big" : "little",
+            TARGET_BYTE_ORDER == BFD_ENDIAN_BIG ? "big" : "little");
 }
 
 

diff --git a/gdb/config/arc/tm-arc.h b/gdb/config/arc/tm-arc.h
index a015843778183dd0da8a06bf91f7b5ed7827eee1..52ed0910437662b5fb57ab5e61db3555e433d018 100644 (file)
--- a/gdb/config/arc/tm-arc.h
+++ b/gdb/config/arc/tm-arc.h
@@ -24,8 +24,8 @@
 /* Used by arc-tdep.c to set the default cpu type.  */
 #define DEFAULT_ARC_CPU_TYPE "base"
 
-/* Byte order is selectable.  */
-#define        TARGET_BYTE_ORDER_SELECTABLE
+/* We have IEEE floating point, if we have any float at all.  */
+#define IEEE_FLOAT (1)
 
 /* Offset from address of function to start of its code.
    Zero on most machines.  */

diff --git a/gdb/config/arm/tm-arm.h b/gdb/config/arm/tm-arm.h
index 80fc1f6582dfe371f8d3633797dd7a909cffb857..cd20548f6f7d37bc6c393e232ef4ebf7451dade4 100644 (file)
--- a/gdb/config/arm/tm-arm.h
+++ b/gdb/config/arm/tm-arm.h
@@ -31,7 +31,6 @@ struct value;
 
 /* Target byte order on ARM defaults to selectable, and defaults to
    little endian.  */
-#define TARGET_BYTE_ORDER_SELECTABLE_P 1
 #define TARGET_BYTE_ORDER_DEFAULT BFD_ENDIAN_LITTLE
 
 /* IEEE format floating point.  */

diff --git a/gdb/config/arm/tm-linux.h b/gdb/config/arm/tm-linux.h
index 97520e701d44000641d280cd0e63e6ae6290512a..51036b663661a708aa650747aa338c1ed87264d4 100644 (file)
--- a/gdb/config/arm/tm-linux.h
+++ b/gdb/config/arm/tm-linux.h
@@ -34,10 +34,6 @@
 extern struct link_map_offsets *arm_linux_svr4_fetch_link_map_offsets (void);
 #define SVR4_FETCH_LINK_MAP_OFFSETS() arm_linux_svr4_fetch_link_map_offsets ()
 
-/* Target byte order on ARM Linux is little endian and not selectable.  */
-#undef TARGET_BYTE_ORDER_SELECTABLE_P
-#define TARGET_BYTE_ORDER_SELECTABLE_P 0
-
 /* Under ARM Linux the traditional way of performing a breakpoint is to
    execute a particular software interrupt, rather than use a particular
    undefined instruction to provoke a trap.  Upon exection of the software

diff --git a/gdb/config/arm/tm-wince.h b/gdb/config/arm/tm-wince.h
index 40e58ba2598a18b11d4cb0580bf1f98b369dec2d..82f97ad7b8ff20b573b8cd48ccaaa6ea2f8f4182 100644 (file)
--- a/gdb/config/arm/tm-wince.h
+++ b/gdb/config/arm/tm-wince.h
@@ -31,8 +31,4 @@
 
 void wince_software_single_step (unsigned int, int);
 
-/* Target byte order is little endian and not selectable on WinCE. */
-#undef  TARGET_BYTE_ORDER_SELECTABLE_P
-#define TARGET_BYTE_ORDER_SELECTABLE_P 0
-
 #endif /* TM_WINCE_H */

diff --git a/gdb/config/mcore/tm-mcore.h b/gdb/config/mcore/tm-mcore.h
index 22631c00f24939ea93fc3958600ccd2909acda73..80e438e185b2137c77febac08a4d1fafef7911d1 100644 (file)
--- a/gdb/config/mcore/tm-mcore.h
+++ b/gdb/config/mcore/tm-mcore.h
@@ -155,9 +155,6 @@ extern void mcore_virtual_frame_pointer (CORE_ADDR, int *, LONGEST *);
 #define TARGET_VIRTUAL_FRAME_POINTER(PC, REGP, OFFP) \
        mcore_virtual_frame_pointer ((PC), (REGP), (OFFP))
 
-/* MCore can be bi-endian. */
-#define TARGET_BYTE_ORDER_SELECTABLE_P 1
-
 /* For PE, gcc will tell us what th real type of
    arguments are when it promotes arguments. */
 #define BELIEVE_PCC_PROMOTION 1

diff --git a/gdb/config/mips/tm-wince.h b/gdb/config/mips/tm-wince.h
index 737ccc60bc570eb81b18918771531f95f3461929..f96c4fabf9a7a53b6436d1953a527c01e4a84602 100644 (file)
--- a/gdb/config/mips/tm-wince.h
+++ b/gdb/config/mips/tm-wince.h
@@ -29,7 +29,6 @@
 #define SOFTWARE_SINGLE_STEP(sig, bp_p) wince_software_single_step (sig, bp_p)
 
 void wince_software_single_step (unsigned int, int);
-#undef TARGET_BYTE_ORDER_SELECTABLE
 #define TARGET_BYTE_ORDER BFD_ENDIAN_LITTLE
 
 #endif /* TM_WINCE_H */

diff --git a/gdb/config/powerpc/tm-ppc-eabi.h b/gdb/config/powerpc/tm-ppc-eabi.h
index 8830504b693f9198c74ee6ee95a2b4a7952e70eb..92e41271fc1e3146037bf598e4c4fbb2facf526d 100644 (file)
--- a/gdb/config/powerpc/tm-ppc-eabi.h
+++ b/gdb/config/powerpc/tm-ppc-eabi.h
@@ -39,8 +39,6 @@
 /* Say that we're using ELF, not XCOFF.  */
 #define ELF_OBJECT_FORMAT 1
 
-#define TARGET_BYTE_ORDER_SELECTABLE_P 1
-
 /* The value of symbols of type N_SO and N_FUN maybe null when 
    it shouldn't be. */
 #define SOFUN_ADDRESS_MAYBE_MISSING

diff --git a/gdb/config/sh/tm-wince.h b/gdb/config/sh/tm-wince.h
index ddebbd2714e79c9c7f68afe4375576a13d16dbf7..e939808c0842d156f078348d2c7c56c13e75c001 100644 (file)
--- a/gdb/config/sh/tm-wince.h
+++ b/gdb/config/sh/tm-wince.h
@@ -28,7 +28,6 @@
 #undef SOFTWARE_SINGLE_STEP
 #define SOFTWARE_SINGLE_STEP(sig, bp_p) wince_software_single_step (sig, bp_p)
 void wince_software_single_step (unsigned int, int);
-#undef TARGET_BYTE_ORDER_SELECTABLE
 #define TARGET_BYTE_ORDER BFD_ENDIAN_LITTLE
 
 #endif /* TM_WINCE_H */

diff --git a/gdb/config/sparc/tm-sparclet.h b/gdb/config/sparc/tm-sparclet.h
index 5382718660e68115cd1d4120d6d0254479a8e241..cc52d5a26d59835961988c51ea456429f87cc339 100644 (file)
--- a/gdb/config/sparc/tm-sparclet.h
+++ b/gdb/config/sparc/tm-sparclet.h
@@ -49,7 +49,6 @@ enum {
 /* overrides of tm-sparc.h */
 
 #undef TARGET_BYTE_ORDER
-#define TARGET_BYTE_ORDER_SELECTABLE
 
 /* Sequence of bytes for breakpoint instruction (ta 1). */
 #undef BREAKPOINT

diff --git a/gdb/config/sparc/tm-sparclite.h b/gdb/config/sparc/tm-sparclite.h
index 71723f2963a0fb73b1d78b0827caec961ac17a19..8fafe5e3b4f4b4547bf208eb651847e95ad01bdf 100644 (file)
--- a/gdb/config/sparc/tm-sparclite.h
+++ b/gdb/config/sparc/tm-sparclite.h
@@ -45,7 +45,6 @@ enum {
 /* overrides of tm-sparc.h */
 
 #undef TARGET_BYTE_ORDER
-#define TARGET_BYTE_ORDER_SELECTABLE
 
 /* Select the sparclite disassembler.  Slightly different instruction set from
    the V8 sparc.  */

diff --git a/gdb/doc/ChangeLog b/gdb/doc/ChangeLog
index f94c1025fcc34b34bee204a24a8f08b10a65396a..2d8bc4bcc46c09c1a31ff5621da94ee04fb6157d 100644 (file)
--- a/gdb/doc/ChangeLog
+++ b/gdb/doc/ChangeLog
@@ -1,3 +1,8 @@
+2002-01-20  Andrew Cagney  <ac131313@redhat.com>
+
+       * gdbint.texinfo (Target Architecture Definition): Delete
+       description of TARGET_BYTE_ORDER_SELECTABLE_P.
+
 2002-01-20  Andrew Cagney  <ac131313@redhat.com>
 
        * gdbint.texinfo (Host Definition): Revise.  xm-xyz.h and xyz.mh

diff --git a/gdb/doc/gdbint.texinfo b/gdb/doc/gdbint.texinfo
index 6605a7683287f567905ad01b8783469815a89da8..3139cf0fea43abd66bc6a6d1b360412d46125c68 100644 (file)
--- a/gdb/doc/gdbint.texinfo
+++ b/gdb/doc/gdbint.texinfo
@@ -3480,12 +3480,6 @@ The ordering of bytes in the target.  This must be either
 @code{BFD_ENDIAN_BIG} or @code{BFD_ENDIAN_LITTLE}.  This macro replaces
 @code{TARGET_BYTE_ORDER} which is deprecated.
 
-@item TARGET_BYTE_ORDER_SELECTABLE_P
-@findex TARGET_BYTE_ORDER_SELECTABLE_P
-Non-zero if the target has both @code{BIG_ENDIAN} and
-@code{BFD_ENDIAN_LITTLE} variants.  This macro replaces
-@code{TARGET_BYTE_ORDER_SELECTABLE} which is deprecated.
-
 @item TARGET_CHAR_BIT
 @findex TARGET_CHAR_BIT
 Number of bits in a char; defaults to 8.

diff --git a/gdb/gdbarch.h b/gdb/gdbarch.h
index 1c09facd39911a2640a991feae90e76e18b9b21a..d3b6170fb973aa5b718370212c2b8de58e539b03 100644 (file)
--- a/gdb/gdbarch.h
+++ b/gdb/gdbarch.h
@@ -2392,33 +2392,7 @@ extern void register_gdbarch_swap (void *data, unsigned long size, gdbarch_swap_
 
 /* The target-system-dependent byte order is dynamic */
 
-/* TARGET_BYTE_ORDER_SELECTABLE_P determines if the target endianness
-   is selectable at runtime.  The user can use the ``set endian''
-   command to change it.  TARGET_BYTE_ORDER_AUTO is nonzero when
-   target_byte_order should be auto-detected (from the program image
-   say). */
-
-#if GDB_MULTI_ARCH
-/* Multi-arch GDB is always bi-endian. */
-#define TARGET_BYTE_ORDER_SELECTABLE_P 1
-#endif
-
-#ifndef TARGET_BYTE_ORDER_SELECTABLE_P
-/* compat - Catch old targets that define TARGET_BYTE_ORDER_SLECTABLE
-   when they should have defined TARGET_BYTE_ORDER_SELECTABLE_P 1 */
-#ifdef TARGET_BYTE_ORDER_SELECTABLE
-#define TARGET_BYTE_ORDER_SELECTABLE_P 1
-#else
-#define TARGET_BYTE_ORDER_SELECTABLE_P 0
-#endif
-#endif
-
 extern int target_byte_order;
-#ifdef TARGET_BYTE_ORDER_SELECTABLE
-/* compat - Catch old targets that define TARGET_BYTE_ORDER_SELECTABLE
-   and expect defs.h to re-define TARGET_BYTE_ORDER. */
-#undef TARGET_BYTE_ORDER
-#endif
 #ifndef TARGET_BYTE_ORDER
 #define TARGET_BYTE_ORDER (target_byte_order + 0)
 #endif

diff --git a/gdb/gdbarch.sh b/gdb/gdbarch.sh
index d6046ec4b88fc781b1fc936aa050a4b8dc779762..23998d5c9a6c15673ebbb7cb430531aedde06960 100755 (executable)
--- a/gdb/gdbarch.sh
+++ b/gdb/gdbarch.sh
@@ -1056,33 +1056,7 @@ extern void register_gdbarch_swap (void *data, unsigned long size, gdbarch_swap_
 
 /* The target-system-dependent byte order is dynamic */
 
-/* TARGET_BYTE_ORDER_SELECTABLE_P determines if the target endianness
-   is selectable at runtime.  The user can use the \`\`set endian''
-   command to change it.  TARGET_BYTE_ORDER_AUTO is nonzero when
-   target_byte_order should be auto-detected (from the program image
-   say). */
-
-#if GDB_MULTI_ARCH
-/* Multi-arch GDB is always bi-endian. */
-#define TARGET_BYTE_ORDER_SELECTABLE_P 1
-#endif
-
-#ifndef TARGET_BYTE_ORDER_SELECTABLE_P
-/* compat - Catch old targets that define TARGET_BYTE_ORDER_SLECTABLE
-   when they should have defined TARGET_BYTE_ORDER_SELECTABLE_P 1 */
-#ifdef TARGET_BYTE_ORDER_SELECTABLE
-#define TARGET_BYTE_ORDER_SELECTABLE_P 1
-#else
-#define TARGET_BYTE_ORDER_SELECTABLE_P 0
-#endif
-#endif
-
 extern int target_byte_order;
-#ifdef TARGET_BYTE_ORDER_SELECTABLE
-/* compat - Catch old targets that define TARGET_BYTE_ORDER_SELECTABLE
-   and expect defs.h to re-define TARGET_BYTE_ORDER. */
-#undef TARGET_BYTE_ORDER
-#endif
 #ifndef TARGET_BYTE_ORDER
 #define TARGET_BYTE_ORDER (target_byte_order + 0)
 #endif

diff --git a/gdb/gdbserver/low-sim.c b/gdb/gdbserver/low-sim.c
index e98b6c54371771e618e4d1ac46a327934cebb68e..86a040621d9f5293c996a5e374c5bcd0ff83035d 100644 (file)
--- a/gdb/gdbserver/low-sim.c
+++ b/gdb/gdbserver/low-sim.c
@@ -88,10 +88,8 @@ create_inferior (char *program, char **argv)
 {
   bfd *abfd;
   int pid = 0;
-#ifdef TARGET_BYTE_ORDER_SELECTABLE
   char **new_argv;
   int nargs;
-#endif
 
   abfd = bfd_openr (program, 0);
   if (!abfd)
@@ -108,7 +106,6 @@ create_inferior (char *program, char **argv)
       exit (1);
     }
 
-#ifdef TARGET_BYTE_ORDER_SELECTABLE
   /* Add "-E big" or "-E little" to the argument list depending on the
      endianness of the program to be loaded.  */
   for (nargs = 0; argv[nargs] != NULL; nargs++)                /* count the args */
@@ -120,7 +117,6 @@ create_inferior (char *program, char **argv)
   new_argv[nargs + 1] = bfd_big_endian (abfd) ? "big" : "little";
   new_argv[nargs + 2] = NULL;
   argv = new_argv;
-#endif
 
   /* Create an instance of the simulator.  */
   default_callback.init (&default_callback);

diff --git a/gdb/remote-sim.c b/gdb/remote-sim.c
index dbae403f8dc448018465c5184e4be482adc46e6d..1d037c8fa79d777e410be702fd57a9b04447c8a9 100644 (file)
--- a/gdb/remote-sim.c
+++ b/gdb/remote-sim.c
@@ -474,8 +474,7 @@ gdbsim_open (char *args, int from_tty)
   strcpy (arg_buf, "gdbsim");  /* 7 */
   /* Specify the byte order for the target when it is both selectable
      and explicitly specified by the user (not auto detected). */
-  if (TARGET_BYTE_ORDER_SELECTABLE_P
-      && !TARGET_BYTE_ORDER_AUTO)
+  if (!TARGET_BYTE_ORDER_AUTO)
     {
       switch (TARGET_BYTE_ORDER)
        {

diff --git a/gdb/sparc-tdep.c b/gdb/sparc-tdep.c
index a32841c99258e946791d1adec00202bfadececa4..1b7954b2da7bcbe40f4c4d38a524fbdf6164f1ee 100644 (file)
--- a/gdb/sparc-tdep.c
+++ b/gdb/sparc-tdep.c
@@ -2200,15 +2200,8 @@ sparc_target_architecture_hook (const bfd_arch_info_type *ap)
 
   if (ap->mach == bfd_mach_sparc_sparclite_le)
     {
-      if (TARGET_BYTE_ORDER_SELECTABLE_P)
-       {
-         target_byte_order = BFD_ENDIAN_LITTLE;
-         bi_endian = 1;
-       }
-      else
-       {
-         warning ("This GDB does not support little endian sparclite.");
-       }
+      target_byte_order = BFD_ENDIAN_LITTLE;
+      bi_endian = 1;
     }
   else
     bi_endian = 0;