Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Pedro Alves <pedro@codesourcery.com>
Cc: gdb-patches@sourceware.org, hjl.tools@gmail.com, msnyder@vmware.com
Subject: Re: PATCH: Support x86 pseudo registers
Date: Fri, 12 Mar 2010 16:04:00 -0000	[thread overview]
Message-ID: <83iq9134rm.fsf@gnu.org> (raw)
In-Reply-To: <201003121526.37646.pedro@codesourcery.com>

> From: Pedro Alves <pedro@codesourcery.com>
> Date: Fri, 12 Mar 2010 15:26:37 +0000
> Cc: hjl.tools@gmail.com,
>  msnyder@vmware.com
> 
> > > I just realized that this change means that $sp is now just
> > > a 16-bit word of $esp, instead of a pseudo-register resolving to
> > > either $esp/$rsp (32-bit/64-bit).  I can't say it is actually wrong to
> > > have it that way
> > 
> > I think it's very wrong, because it means we no longer have a generic
> > stack pointer register, at least not on x86.  Is that true?
> 
> A certainly agree very much that it's not convenient to have
> $sp not be the largest stack pointer.  I said the above based on:
> 
>  @cindex standard registers
>  @value{GDBN} has four ``standard'' register names that are available (in
>  expressions) on most machines---whenever they do not conflict with an
>                                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>  architecture's canonical mnemonics for registers.  The register names
>  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>  @code{$pc} and @code{$sp} are used for the program counter register and
>  the stack pointer.  @code{$fp} is used for a register that contains a
>  pointer to the current stack frame, and @code{$ps} is used for a
>  register that contains the processor status.  For example,
>  you could print the program counter in hex with
> 
> So, should that sentence of the manual be relaxed?

Maybe, but frankly I don't really understand what it says, exactly.
Does it mean that if the name does clash with the architecture, the
architecture's meaning is used?

Anyway, are there any such conflicts in the current codebase?

> I guess this would be a good place to at least mention the x86 $sp
> is always $esp or $rsp.

Yes, I think so.


  reply	other threads:[~2010-03-12 16:04 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-01 17:02 H.J. Lu
2010-03-02 11:17 ` Mark Kettenis
2010-03-02 13:55 ` Daniel Jacobowitz
2010-03-02 14:08   ` H.J. Lu
2010-03-02 15:04     ` Daniel Jacobowitz
2010-03-02 15:37       ` H.J. Lu
2010-03-02 17:54       ` Eli Zaretskii
2010-03-02 19:01         ` H.J. Lu
2010-03-02 20:54           ` Eli Zaretskii
2010-03-02 21:06             ` H.J. Lu
2010-03-02 21:47               ` Eli Zaretskii
2010-03-02 21:52                 ` Daniel Jacobowitz
2010-03-02 21:57                   ` Michael Snyder
2010-03-02 22:00                     ` H.J. Lu
2010-03-02 22:07                       ` H.J. Lu
2010-03-03 17:33                         ` Eli Zaretskii
2010-03-03 17:53                           ` H.J. Lu
2010-03-03 18:09                             ` Eli Zaretskii
2010-03-03 20:20                               ` H.J. Lu
2010-03-12  4:23                           ` Pedro Alves
2010-03-12  5:32                             ` H.J. Lu
2010-03-12  6:07                               ` H.J. Lu
2010-03-12 18:22                               ` Michael Snyder
2010-03-12  8:30                             ` Eli Zaretskii
2010-03-12 15:26                               ` Pedro Alves
2010-03-12 16:04                                 ` Eli Zaretskii [this message]
2010-03-12 16:30                                   ` Daniel Jacobowitz
2010-03-12 18:14                                     ` Eli Zaretskii
2010-03-12 18:20                                       ` Pedro Alves
2010-03-12 18:55                                         ` Eli Zaretskii
2010-03-12 19:47                                           ` H.J. Lu
2010-03-12 16:31                                   ` Pedro Alves
2010-03-02 21:58                   ` H.J. Lu

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=83iq9134rm.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=gdb-patches@sourceware.org \
    --cc=hjl.tools@gmail.com \
    --cc=msnyder@vmware.com \
    --cc=pedro@codesourcery.com \
    /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