This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[PATCH] Fix comment related to previous patch to sparc64-tdep.c


Cmmitted.


Index: ChangeLog
from  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.

Index: sparc64-tdep.c
===================================================================
RCS file: /cvs/src/src/gdb/sparc64-tdep.c,v
retrieving revision 1.6
diff -u -p -r1.6 sparc64-tdep.c
--- sparc64-tdep.c 3 Jan 2004 20:46:30 -0000 1.6
+++ sparc64-tdep.c 3 Jan 2004 23:24:07 -0000
@@ -674,8 +674,10 @@ sparc64_store_floating_fields (struct re
          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));


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]