Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* Floating point types in target descriptions
@ 2007-04-13 12:48 Paul Brook
  2007-04-13 13:53 ` Daniel Jacobowitz
  0 siblings, 1 reply; 3+ messages in thread
From: Paul Brook @ 2007-04-13 12:48 UTC (permalink / raw)
  To: gdb-patches

The patch below adds ieee_single and ieee_double predefined types for use in 
XML target descriptions.  These are needed to describe vector types with 
floating point elements.

I considered using "float"/"double" and/or "float32"/"float64". However the 
former are ambiguous with the "float" register type, and the latter don't 
really tell you want you're getting. If targets need non-ieee types they can 
add them like arm_fpa_ext.

Ok?

Paul

2007-04-13  Paul Brook  <paul@codesourcery.com>

	* target-descriptions.c (tdesc_named_type): Add ieee_single and
	ieee_double.
	* doc/gdb.texinfo: Document ieee_single and ieee_double target types.

Index: gdb/doc/gdb.texinfo
===================================================================
--- gdb/doc/gdb.texinfo	(revision 168411)
+++ gdb/doc/gdb.texinfo	(working copy)
@@ -26070,6 +26070,12 @@ pointers; printing a code pointer conver
 address.  The stack pointer and any dedicated address registers
 may be marked as data pointers.
 
+@item ieee_single
+Single precision IEEE floating point.
+
+@item ieee_double
+Double precision IEEE floating point.
+
 @item arm_fpa_ext
 The 12-byte extended precision format used by ARM FPA registers.
 
Index: gdb/target-descriptions.c
===================================================================
--- gdb/target-descriptions.c	(revision 168411)
+++ gdb/target-descriptions.c	(working copy)
@@ -409,6 +409,12 @@ tdesc_named_type (const struct tdesc_fea
   if (strcmp (id, "uint64") == 0)
     return builtin_type_uint64;
 
+  if (strcmp (id, "ieee_single") == 0)
+    return builtin_type_ieee_single;
+
+  if (strcmp (id, "ieee_double") == 0)
+    return builtin_type_ieee_double;
+
   if (strcmp (id, "arm_fpa_ext") == 0)
     return builtin_type_arm_ext;
 


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Floating point types in target descriptions
  2007-04-13 12:48 Floating point types in target descriptions Paul Brook
@ 2007-04-13 13:53 ` Daniel Jacobowitz
  2007-04-14  0:32   ` Paul Brook
  0 siblings, 1 reply; 3+ messages in thread
From: Daniel Jacobowitz @ 2007-04-13 13:53 UTC (permalink / raw)
  To: Paul Brook; +Cc: gdb-patches

On Fri, Apr 13, 2007 at 01:48:09PM +0100, Paul Brook wrote:
> The patch below adds ieee_single and ieee_double predefined types for use in 
> XML target descriptions.  These are needed to describe vector types with 
> floating point elements.
> 
> I considered using "float"/"double" and/or "float32"/"float64". However the 
> former are ambiguous with the "float" register type, and the latter don't 
> really tell you want you're getting. If targets need non-ieee types they can 
> add them like arm_fpa_ext.
> 
> Ok?

Thanks, this is OK.

-- 
Daniel Jacobowitz
CodeSourcery


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Floating point types in target descriptions
  2007-04-13 13:53 ` Daniel Jacobowitz
@ 2007-04-14  0:32   ` Paul Brook
  0 siblings, 0 replies; 3+ messages in thread
From: Paul Brook @ 2007-04-14  0:32 UTC (permalink / raw)
  To: Daniel Jacobowitz; +Cc: gdb-patches

On Friday 13 April 2007 14:53, Daniel Jacobowitz wrote:
> On Fri, Apr 13, 2007 at 01:48:09PM +0100, Paul Brook wrote:
> > The patch below adds ieee_single and ieee_double predefined types for use
> > in XML target descriptions.  These are needed to describe vector types
> > with floating point elements.
>
> Thanks, this is OK.

Applied to CVS head.

Paul


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2007-04-13 20:33 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-04-13 12:48 Floating point types in target descriptions Paul Brook
2007-04-13 13:53 ` Daniel Jacobowitz
2007-04-14  0:32   ` Paul Brook

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox