* sparc64-tdep.c (sparc64_store_floating_fields): Update comment
authorMark Kettenis <kettenis@gnu.org>
Sat, 3 Jan 2004 23:24:57 +0000 (23:24 +0000)
committerMark Kettenis <kettenis@gnu.org>
Sat, 3 Jan 2004 23:24:57 +0000 (23:24 +0000)
such that it mentions a specific version of GCC that exhibits this
bug.

gdb/ChangeLog
gdb/sparc64-tdep.c

index 65db16f443e63be871fa688e1a495741d4692df7..a2bbed57a7e65d9f26cb51728e48b6dc8ea5c0b5 100644 (file)
@@ -1,3 +1,9 @@
+2004-01-04  Mark Kettenis  <kettenis@gnu.org>
+
+       * sparc64-tdep.c (sparc64_store_floating_fields): Update comment
+       such that it mentions a specific version of GCC that exhibits this
+       bug.
+
 2004-01-03  Mark Kettenis  <kettenis@gnu.org>
 
        * sparc64-tdep.c (sparc64_store_floating_fields): If TYPE is a
index bf3aeca1dbde9c76537f609d46cffe7cf5e5c461..7e7079181741eee67a347940abb25eddde7f3aed 100644 (file)
@@ -674,8 +674,10 @@ sparc64_store_floating_fields (struct regcache *regcache, struct type *type,
          at all, but rather as an ordinary `float' argument.  This
          argument will be stored in %f1, as required by the psABI.
          However, as a member of a structure the psABI requires it to
-         be stored in.  To appease GCC, if a structure has only a
-         single `float' member, we store its value in %f1 too.  */
+         be stored in %f0.  This bug is present in GCC 3.3.2, but
+         probably in older releases to.  To appease GCC, if a
+         structure has only a single `float' member, we store its
+         value in %f1 too (we already have stored in %f0).  */
       if (TYPE_NFIELDS (type) == 1)
        {
          struct type *subtype = check_typedef (TYPE_FIELD_TYPE (type, 0));