public inbox for gdb-cvs@sourceware.org
help / color / mirror / Atom feed
From: Jan Vrany <jv@sourceware.org>
To: gdb-cvs@sourceware.org
Subject: [binutils-gdb] ppc: use "trap" ("tw, 31, 0, 0") as breakpoint instruction
Date: Wed,  8 Dec 2021 11:02:06 +0000 (GMT)	[thread overview]
Message-ID: <20211208110206.EF3DF3858D28@sourceware.org> (raw)

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=44f3c3c212f9b568a6b3cff0444b6750e3d8680f

commit 44f3c3c212f9b568a6b3cff0444b6750e3d8680f
Author: Jan Vrany <jan.vrany@labware.com>
Date:   Wed Dec 8 10:46:49 2021 +0000

    ppc: use "trap" ("tw, 31, 0, 0") as breakpoint instruction
    
    Power ISA 3.0 B spec [1], sections 3.3.11 "Fixed-Point Trap Instructions"
    and section C.6 "Trap Mnemonics" specify "tw, 31, 0, 0" (encoded as
    0x7fe00008) as canonical unconditional trap instruction.
    
    This commit changes the breakpoint instruction used by GDB from
    "tw 12, r2, r2" to unconditional "trap".
    
    [1]: https://openpowerfoundation.org/?resource_lib=power-isa-version-3-0

Diff:
---
 gdb/rs6000-tdep.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gdb/rs6000-tdep.c b/gdb/rs6000-tdep.c
index 87a494e0bb8..43880fa4426 100644
--- a/gdb/rs6000-tdep.c
+++ b/gdb/rs6000-tdep.c
@@ -824,8 +824,8 @@ rs6000_fetch_pointer_argument (struct frame_info *frame, int argi,
 
 /* Sequence of bytes for breakpoint instruction.  */
 
-constexpr gdb_byte big_breakpoint[] = { 0x7d, 0x82, 0x10, 0x08 };
-constexpr gdb_byte little_breakpoint[] = { 0x08, 0x10, 0x82, 0x7d };
+constexpr gdb_byte big_breakpoint[] = { 0x7f, 0xe0, 0x00, 0x08 };
+constexpr gdb_byte little_breakpoint[] = { 0x08, 0x00, 0xe0, 0x7f };
 
 typedef BP_MANIPULATION_ENDIAN (little_breakpoint, big_breakpoint)
   rs6000_breakpoint;


                 reply	other threads:[~2021-12-08 11:02 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

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=20211208110206.EF3DF3858D28@sourceware.org \
    --to=jv@sourceware.org \
    --cc=gdb-cvs@sourceware.org \
    /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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).