* config/i386/xm-i386.h (HOST_BYTE_ORDER): Removed.

[deliverable/binutils-gdb.git] / gdb / i387-tdep.c

diff --git a/gdb/i387-tdep.c b/gdb/i387-tdep.c
index b7a9554d218fa5a0337a703731a4b28b5b2a6097..275a1e291e22fea829bb2a004649912287dbcffa 100644 (file)
--- a/gdb/i387-tdep.c
+++ b/gdb/i387-tdep.c
@@ -27,6 +27,8 @@
 #include "gdbcore.h"
 #include "floatformat.h"
 #include "regcache.h"
+#include "gdb_assert.h"
+#include "doublest.h"
 
 
 /* FIXME: Eliminate the next two functions when we have the time to
@@ -160,22 +162,25 @@ static void
 print_i387_value (char *raw)
 {
   DOUBLEST value;
+  int len = TARGET_LONG_DOUBLE_BIT / TARGET_CHAR_BIT;
+  char *tmp = alloca (len);
 
-  /* Avoid call to floatformat_to_doublest if possible to preserve as
-     much information as possible.  */
+  /* This code only works on targets where ... */
+  gdb_assert (TARGET_LONG_DOUBLE_FORMAT == &floatformat_i387_ext);
 
-#ifdef HAVE_LONG_DOUBLE
-  if (sizeof (value) == sizeof (long double)
-      && HOST_LONG_DOUBLE_FORMAT == &floatformat_i387_ext)
-    {
-      /* Copy straight over, but take care of the padding.  */
-      memcpy (&value, raw, FPU_REG_RAW_SIZE);
-      memset ((char *) &value + FPU_REG_RAW_SIZE, 0,
-             sizeof (value) - FPU_REG_RAW_SIZE);
-    }
-  else
-#endif
-    floatformat_to_doublest (&floatformat_i387_ext, raw, &value);
+  /* Take care of the padding.  FP reg is 80 bits.  The same value in
+     memory is 96 bits.  */
+  gdb_assert (FPU_REG_RAW_SIZE < len);
+  memcpy (tmp, raw, FPU_REG_RAW_SIZE);
+  memset (tmp + FPU_REG_RAW_SIZE, 0, len - FPU_REG_RAW_SIZE);
+  
+  /* Extract the value as a DOUBLEST.  */
+  /* Use extract_floating() rather than floatformat_to_doublest().
+     The latter is lossy in nature.  Once GDB gets a host/target
+     independent and non-lossy FP it will become possible to bypass
+     extract_floating() and call floatformat*() directly.  Note also
+     the assumptions about TARGET_LONG_DOUBLE above.  */
+  value = extract_floating (tmp, len);
 
   /* We try to print 19 digits.  The last digit may or may not contain
      garbage, but we'd better print one too many.  We need enough room
@@ -255,7 +260,7 @@ print_i387_status_word (unsigned int status)
   puts_filtered ("  ");
   printf_filtered (" %s", (status & 0x0080) ? "ES" : "  ");
   puts_filtered ("  ");
-  printf_filtered (" %s", (status & 0x0080) ? "SF" : "  ");
+  printf_filtered (" %s", (status & 0x0040) ? "SF" : "  ");
   puts_filtered ("  ");
   printf_filtered (" %s", (status & 0x0100) ? "C0" : "  ");
   printf_filtered (" %s", (status & 0x0200) ? "C1" : "  ");