From 367957b7cada78f80c572ba7668004782f59ff8e Mon Sep 17 00:00:00 2001 From: Andrew Cagney Date: Wed, 4 Jul 2001 19:19:58 +0000 Subject: * i387-tdep.c: Include "gdb_assert.h". (print_i387_value): Use extract_floating to extract the FP value from a zero padded local buffer. --- gdb/ChangeLog | 6 ++++++ gdb/i387-tdep.c | 32 ++++++++++++++++++-------------- 2 files changed, 24 insertions(+), 14 deletions(-) (limited to 'gdb') diff --git a/gdb/ChangeLog b/gdb/ChangeLog index d6e761ac315..f0dcbeefc08 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +2001-06-29 Andrew Cagney + + * i387-tdep.c: Include "gdb_assert.h". + (print_i387_value): Use extract_floating to extract the FP value + from a zero padded local buffer. + 2001-06-28 Andrew Cagney * TODO: Delete all thread items. The thread code was overhauled. diff --git a/gdb/i387-tdep.c b/gdb/i387-tdep.c index e221b0c9e73..cc403fcc54c 100644 --- a/gdb/i387-tdep.c +++ b/gdb/i387-tdep.c @@ -27,6 +27,7 @@ #include "gdbcore.h" #include "floatformat.h" #include "regcache.h" +#include "gdb_assert.h" /* FIXME: Eliminate the next two functions when we have the time to @@ -160,22 +161,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 -- cgit v1.2.1