diff options
| author | Andrew Cagney <cagney@redhat.com> | 2001-07-04 19:19:58 +0000 |
|---|---|---|
| committer | Andrew Cagney <cagney@redhat.com> | 2001-07-04 19:19:58 +0000 |
| commit | 367957b7cada78f80c572ba7668004782f59ff8e (patch) | |
| tree | 9008bd4c2ed0caf78605aa875091d28ad6194627 | |
| parent | 51d41a245011545e47bdd57bb745d1e5cdd876a9 (diff) | |
| download | gdb-367957b7cada78f80c572ba7668004782f59ff8e.tar.gz | |
* i387-tdep.c: Include "gdb_assert.h".
(print_i387_value): Use extract_floating to extract the FP value
from a zero padded local buffer.
| -rw-r--r-- | gdb/ChangeLog | 6 | ||||
| -rw-r--r-- | gdb/i387-tdep.c | 32 |
2 files changed, 24 insertions, 14 deletions
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 <ac131313@redhat.com> + + * 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 <ac131313@redhat.com> * 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 |