Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Daniel Jacobowitz <drow@false.org>
To: Thiago Jung Bauermann <bauerman@br.ibm.com>,
		Mark Kettenis <mark.kettenis@xs4all.nl>
Cc: Luis Machado <luisgpm@linux.vnet.ibm.com>, gdb-patches@sourceware.org
Subject: Re: Printing decimal128 types out of registers
Date: Mon, 21 Jan 2008 17:54:00 -0000	[thread overview]
Message-ID: <20080121175413.GA25254@caradoc.them.org> (raw)
In-Reply-To: <200801211730.m0LHUGbu021315@brahms.sibelius.xs4all.nl> <1200927274.32125.36.camel@localhost.localdomain>

On Mon, Jan 21, 2008 at 12:54:34PM -0200, Thiago Jung Bauermann wrote:
> Yes, even for software decimal float implementations the calling
> convention for PowerPC. is to put _Decimal128 in a pair of float
> registers

This, by itself, does not mean we need to be able to display two
registers as a _Decimal128.  We will still be able to display a named
argument in two registers as _Decimal128, because the debug
information will (presumably) show it as two pieces in two registers
using DW_OP_piece.

There's no way to do math on a _Decimal128 value in two FP registers
without the hardware support.  So I suggest we not have $dl0 except on
hardware where the GDB user might look at the next instruction, see
that it operates on register dl0, and want to check the value of that
register.

> I was talking to Luis about this, and he suggested an approach other
> than using pseudo registers: create a way to make GDB consider two
> consecutive registers as contiguous data. Since there are other types
> that are also passed in consecutive registers (e.g., long long,
> soft-float, IBM lon double), this mechanism would be more useful. I
> think this is a good idea.

This is like stepping backwards in time.  We used to treat consecutive
registers as a big data blob and it was a terrible mess ... if you
want to examine typed multi-register values, I suggest leaving any
"consecutive" requirement out of the picture.

On Mon, Jan 21, 2008 at 06:30:16PM +0100, Mark Kettenis wrote:
> > Perhaps some syntax could be created, like:
> > 
> > (gdb) print {_Decimal128} $f2,$f3
> > $1 = 1.2
> > 
> > What do you think?
> 
> I'm not sure how this would fit into the expression parser.  t might
> get really ugly.

Definitely agreed.  Maybe an explicit command to create a name for a
group of registers in order, using the same underlying mechanism we
use for DWARF.

-- 
Daniel Jacobowitz
CodeSourcery


  parent reply	other threads:[~2008-01-21 17:54 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-11-07 19:33 [PATCH] PPC - Printing Decimal 128 " Luis Machado
2007-11-26 21:40 ` Luis Machado
2007-12-26 12:42   ` [PING] " Luis Machado
2008-01-17 19:03 ` Printing decimal128 " Luis Machado
2008-01-17 19:36   ` Thiago Jung Bauermann
2008-01-18 15:43     ` Luis Machado
2008-01-18 16:12       ` Mark Kettenis
2008-01-18 16:38         ` Luis Machado
2008-01-18 17:20           ` Mark Kettenis
2008-01-18 18:52             ` Luis Machado
2008-01-18 19:54               ` Mark Kettenis
2008-01-19  0:04       ` Daniel Jacobowitz
2008-01-21 14:55         ` Thiago Jung Bauermann
2008-01-21 17:31           ` Mark Kettenis
2008-01-23 15:39             ` Thiago Jung Bauermann
2008-01-21 17:54     ` Daniel Jacobowitz [this message]
2008-01-23 15:11       ` Thiago Jung Bauermann
2008-01-23 15:20         ` Daniel Jacobowitz
2008-01-23 16:56           ` Thiago Jung Bauermann
2008-01-30 15:46           ` Thiago Jung Bauermann
2008-01-30 16:45             ` Thiago Jung Bauermann
2008-01-30 18:27             ` Mark Kettenis
2008-01-30 18:38               ` Thiago Jung Bauermann
2008-01-30 20:28                 ` Mark Kettenis
2008-01-30 21:26                   ` Thiago Jung Bauermann
2008-01-30 21:39                     ` Ulrich Weigand
2008-01-31 14:47                       ` Thiago Jung Bauermann
2008-01-31 15:06                         ` Ulrich Weigand
2008-01-30 21:42                     ` Mark Kettenis
2008-01-30 23:03                     ` Eli Zaretskii
2008-01-31 15:44                       ` Thiago Jung Bauermann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20080121175413.GA25254@caradoc.them.org \
    --to=drow@false.org \
    --cc=bauerman@br.ibm.com \
    --cc=gdb-patches@sourceware.org \
    --cc=luisgpm@linux.vnet.ibm.com \
    --cc=mark.kettenis@xs4all.nl \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox