Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Ezra Sitorus <ezra.sitorus@arm.com>
To: Luis <luis.machado.foss@gmail.com>
Cc: <gdb-patches@sourceware.org>, <pedro@palves.net>
Subject: Re: [PATCH 3/4] gdb/aarch64: Display fp8 formats in registers
Date: Tue, 14 Oct 2025 01:56:35 +0100	[thread overview]
Message-ID: <aO2fw1BbqIdL6hP8@D2G2X7XGD1> (raw)
In-Reply-To: <148a9787-fe74-4ebb-99dd-8a50174510ce@gmail.com>

On Sat, Oct 11, 2025 at 02:10:11PM +0100, Luis wrote:
> Hi Ezra,
> 
> Given Pedro's input about the representation of fp8's, I'm not sure this is
> the right path. Enabling these types for the registers would also make the
> types visible to the rest of GDB. And users can operate on them via
> arithmetic operators etc.
> 
> Being integers, the ending result wouldn�t be correct.
> 
> It might be viable to explore the python extension option to print the types
> accordingly. We already make vector fields (neon or sve) available as 8, 16,
> 32, 64 and 128 integers. Converting from that would therefore be easy.
> 

Fair! I think a python pretty printer is a good idea. However, would a pretty
belong in the gdb project or the gcc project? The only examples in gdb are from
testcases. The gdb docs references gdb.libstdcxx.v6, but this exists in gcc. In
either scenario, I'm not too sure where the actual files would go to.

> Also, I'm not too keen on expanding even further these neon/sve union types.
> They are very hard to parse due to the multitude of representations. Ideally
> we�d put together some pseudo-registers to do the job. That would also help
> with remote debugging stub target description compatibility, as those
> wouldn�t need to expose these internal details of gdb.

Your previous comment seems to suggest adding a pretty printer for the registers
too but this one seems to suggest adding pseudo-registers instead. For pseudo-regs,
would the approach be something like putting the new float formats in aarch64-tdep.c,
and assigning the new pseudo regs' types to be the fp8 float formats? 

Thanks!

Ezra

  reply	other threads:[~2025-10-14  0:57 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-25 21:18 [PATCH 0/4] gdb: Display fp8 format for AArch64 registers Ezra.Sitorus
2025-09-25 21:18 ` [PATCH 1/4] libiberty: Add fp8 float formats Ezra.Sitorus
2025-09-25 21:18 ` [PATCH] gdb: Add support for fp8 formats Ezra.Sitorus
2025-09-25 21:18 ` [PATCH 3/4] gdb/aarch64: Display fp8 formats in registers Ezra.Sitorus
2025-10-11 13:10   ` Luis
2025-10-14  0:56     ` Ezra Sitorus [this message]
2025-10-17 10:20       ` Luis
2025-09-25 21:18 ` [PATCH 4/4] gdb/aarch64: Tests for verifying fp8 register formats Ezra.Sitorus
2025-09-29 17:55 ` [PATCH 0/4] gdb: Display fp8 format for AArch64 registers Pedro Alves

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=aO2fw1BbqIdL6hP8@D2G2X7XGD1 \
    --to=ezra.sitorus@arm.com \
    --cc=gdb-patches@sourceware.org \
    --cc=luis.machado.foss@gmail.com \
    --cc=pedro@palves.net \
    /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