From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca by simark.ca with LMTP id oUmFBLMlMGpPfggAWB0awg (envelope-from ) for ; Mon, 15 Jun 2026 12:17:55 -0400 Authentication-Results: simark.ca; dkim=pass (1024-bit key; unprotected) header.d=redhat.com header.i=@redhat.com header.a=rsa-sha256 header.s=mimecast20190719 header.b=AiOfDHZP; dkim-atps=neutral Received: by simark.ca (Postfix, from userid 112) id 0963A1E0A6; Mon, 15 Jun 2026 12:17:55 -0400 (EDT) X-Spam-Checker-Version: SpamAssassin 4.0.1 (2024-03-25) on simark.ca X-Spam-Level: X-Spam-Status: No, score=-6.4 required=5.0 tests=ARC_SIGNED,ARC_VALID,BAYES_00, DKIMWL_WL_HIGH,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI, RCVD_IN_DNSWL_MED autolearn=ham autolearn_force=no version=4.0.1 Received: from vm01.sourceware.org (vm01.sourceware.org [IPv6:2620:52:6:3111::32]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange x25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPS id 1F0A01E070 for ; Mon, 15 Jun 2026 12:17:53 -0400 (EDT) Received: from vm01.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 51D2E4AA3940 for ; Mon, 15 Jun 2026 16:17:51 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 51D2E4AA3940 Authentication-Results: sourceware.org; dkim=pass (1024-bit key, unprotected) header.d=redhat.com header.i=@redhat.com header.a=rsa-sha256 header.s=mimecast20190719 header.b=AiOfDHZP Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by sourceware.org (Postfix) with ESMTP id 59A2D49002D1 for ; Mon, 15 Jun 2026 16:15:54 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 59A2D49002D1 Authentication-Results: sourceware.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=redhat.com ARC-Filter: OpenARC Filter v1.0.0 sourceware.org 59A2D49002D1 Authentication-Results: sourceware.org; arc=none smtp.remote-ip=170.10.129.124 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1781540154; cv=none; b=dBj/MdfR0JTud5tV0WebQPAoWB+E2p7B7gVv63gr/lGRgGrmR5oHYFAJXOz+h3lDg0BI1rGBHXlHEArugcLjdFX9ofii1J52Eyky63HjlOJoQ+jSq5zx5olGlrsuJ3wKXk5+XjYwwhzNpYNWGvIlPb3+kCXVl9RBBm+HZKp4EIw= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1781540154; c=relaxed/simple; bh=vFV4EO0Yt24Ad3LAA8gmqlFcVvJ+2vjcdWO1IiPZX9c=; h=DKIM-Signature:From:To:Subject:Date:Message-ID:MIME-Version; b=JDsA1ngdqNPgpe27j2iiO7ICVEYQCpaSOt1wxWnLHPy1Xp5MZEnNxEhNbe3LeSYVuyJ6gLQEAlY2p7NxlKi0dS1j+EtYv0+HscpcfZonxBLRTrymbabtEnm38yBH56WWIuRUxd8gO+UzIjR9KdmrVc+aSjXVGk+icE4a5KZQV9I= ARC-Authentication-Results: i=1; sourceware.org; dkim=pass (1024-bit key, unprotected) header.d=redhat.com header.i=@redhat.com header.a=rsa-sha256 header.s=mimecast20190719 header.b=AiOfDHZP DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 59A2D49002D1 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1781540153; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=ZZ1P2W8/QpE/z8W8gBDsmTs3pKTY9mUjRzxpaaxhZeo=; b=AiOfDHZPyimCOaiDOsnsVHEndrisLLbFW+eGrLKg7b+u0bP2weJnIroyVjAd2VYBDlI2c/ GYx9JS8duJoIBWULpt9B4nr1rQdBrrGxA1pd1oMaj8iJBfirQIdW66UpxxNPnG4kdx1D5T 6uMACXReD92FqEQoXNVyV7Uv71/ffs8= Received: from mx-prod-mc-06.mail-002.prod.us-west-2.aws.redhat.com (ec2-35-165-154-97.us-west-2.compute.amazonaws.com [35.165.154.97]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-619-ZwdXp9WMMQ6rVBm6PVnoUw-1; Mon, 15 Jun 2026 12:15:50 -0400 X-MC-Unique: ZwdXp9WMMQ6rVBm6PVnoUw-1 X-Mimecast-MFC-AGG-ID: ZwdXp9WMMQ6rVBm6PVnoUw_1781540148 Received: from mx-prod-int-03.mail-002.prod.us-west-2.aws.redhat.com (mx-prod-int-03.mail-002.prod.us-west-2.aws.redhat.com [10.30.177.12]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mx-prod-mc-06.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id C00C91800259; Mon, 15 Jun 2026 16:15:47 +0000 (UTC) Received: from fedora.redhat.com (unknown [10.96.134.37]) by mx-prod-int-03.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id 3C58D1955BCE; Mon, 15 Jun 2026 16:15:44 +0000 (UTC) From: Guinevere Larsen To: gdb-patches@sourceware.org Cc: Guinevere Larsen , Thiago Jung Bauermann , Christina Schimpe Subject: [PATCH v5 6/8] gdb/record: extract the PC to record_full_instruction Date: Mon, 15 Jun 2026 13:14:58 -0300 Message-ID: <20260615161500.34181-7-guinevere@redhat.com> In-Reply-To: <20260615161500.34181-1-guinevere@redhat.com> References: <20260615161500.34181-1-guinevere@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.0 on 10.30.177.12 X-Mimecast-Spam-Score: 0 X-Mimecast-MFC-PROC-ID: e24l0vCcDWXiqcMzomX7o5HEunmD9XaF6zqOHZUIGPY_1781540148 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: 8bit content-type: text/plain; charset="US-ASCII"; x-default=true X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.30 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: gdb-patches-bounces~public-inbox=simark.ca@sourceware.org This commit makes it so the PC is not saved as part of the record_full_instruction effects, but rather gets a special location. That is because a couple of commands would really benefit from it being easy to find the PC (especially ones from record-btrace that's haven't been implemented to record-full yet, such as the ones in PR record/18059), while also possibly allowing for one fewer resizing of the effect vector (and saving an entire byte in the process). This commit also refactored record_full_read_entry_from_bfd and record_full_write_entry_to_bfd, to make them methods of record_full_reg_entry and record_full_mem_entry, and also creates similar methods for record_full_entry and record_full_instruction. These could be turned into constructors in a future step of c++ification, but it felt like too much change for a single commit. Reviewed-by: Thiago Jung Bauermann Reviewed-By: Christina Schimpe --- gdb/record-full.c | 383 ++++++++++++++++++++++++++++------------------ 1 file changed, 237 insertions(+), 146 deletions(-) diff --git a/gdb/record-full.c b/gdb/record-full.c index 54a0251ae19..ec6d40e97cf 100644 --- a/gdb/record-full.c +++ b/gdb/record-full.c @@ -153,6 +153,14 @@ struct record_full_mem_entry DISABLE_COPY_AND_ASSIGN (record_full_mem_entry); + /* Create a mem_entry from a bfd file, when restoring a recording. */ + static record_full_mem_entry from_bfd (bfd *cbfd, asection* osec, + int *bfd_offset); + + /* Save this mem entry to a bfd file. */ + void to_bfd (const gdb_bfd_ref_ptr &obfd, asection *osec, int *bfd_offset, + gdbarch *gdbarch); + gdb_byte *get_loc () { if (len > sizeof (u.buf)) @@ -271,6 +279,13 @@ struct record_full_reg_entry DISABLE_COPY_AND_ASSIGN (record_full_reg_entry); + /* Create a reg_entry from a bfd file, when restoring a recording. */ + static record_full_reg_entry from_bfd (bfd *cbfd, asection* osec, + int *bfd_offset); + + /* Save this reg entry to a bfd file. */ + void to_bfd (const gdb_bfd_ref_ptr &obfd, asection *osec, int *bfd_offset); + gdb_byte *get_loc () { if (len > sizeof (u.buf)) @@ -333,6 +348,13 @@ class record_full_entry DISABLE_COPY_AND_ASSIGN (record_full_entry); + /* Create a generic entry from a bfd file, when restoring a recording. */ + static void from_bfd (bfd *cbfd, asection* osec, int *bfd_offset); + + /* Save this entry to a bfd file. */ + void to_bfd (const gdb_bfd_ref_ptr &obfd, asection *osec, int *bfd_offset, + gdbarch *gdbarch); + record_full_reg_entry& reg () { gdb_assert (type () == record_full_reg); @@ -393,6 +415,7 @@ class record_full_entry this one; * sigval: Whether the inferior received a signal while the following instruction was being recorded; + * pc: The Program Counter at the start of the instruction; * effects: A list of record_full_entry structures, each of which describing one effect that the instruction has on the inferior. @@ -407,10 +430,18 @@ struct record_full_instruction uint32_t insn_num; std::optional sigval; std::vector effects; + record_full_reg_entry pc; /* Execute the full instruction. As a side effect, set record_full_stop_reason. */ void exec_insn (regcache *regcache); + + /* Create a full recorded instruction from a bfd. */ + static void from_bfd (bfd *cbfd, asection* osec, int *bfd_offset); + + /* Save this instruction to a bfd file. */ + void to_bfd (gdb_bfd_ref_ptr obfd, asection *osec, int *bfd_offset, + gdbarch *gdbarch); }; /* If true, query if PREC cannot record memory @@ -721,7 +752,10 @@ record_full_arch_list_add_reg (struct regcache *regcache, int regnum) regcache->cooked_read (regnum, rec.get_loc ()); - record_full_arch_list_add (rec); + if (regnum == gdbarch_pc_regnum (regcache->arch ())) + record_full_incomplete_instruction.pc = std::move (rec.reg ()); + else + record_full_arch_list_add (rec); return 0; } @@ -877,6 +911,7 @@ static enum target_stop_reason record_full_stop_reason void record_full_instruction::exec_insn (regcache *regcache) { + pc.execute (regcache); for (auto &entry : effects) if (entry.execute (regcache)) record_full_stop_reason = TARGET_STOPPED_BY_WATCHPOINT; @@ -2236,68 +2271,89 @@ netorder32 (uint32_t input) return ret; } -static void -record_full_read_entry_from_bfd (bfd *cbfd, asection *osec, int *bfd_offset) +record_full_reg_entry +record_full_reg_entry::from_bfd (bfd *cbfd, asection *osec, int *bfd_offset) { - uint8_t rectype; - uint32_t regnum, len; - uint64_t addr; + uint32_t regnum; regcache *cache = get_thread_regcache (inferior_thread ()); + /* Get register number to regnum. */ + bfdcore_read (cbfd, osec, ®num, sizeof (regnum), + bfd_offset); + regnum = netorder32 (regnum); + + record_full_reg_entry reg (cache->arch (), regnum); + + /* Get val. */ + bfdcore_read (cbfd, osec, reg.get_loc (), + reg.len, bfd_offset); + + if (record_debug) + gdb_printf (gdb_stdlog, + " Reading register %d (1 " + "plus %lu plus %d bytes)\n", + reg.num, + (unsigned long) sizeof (regnum), + reg.len); + return reg; + +} + +record_full_mem_entry +record_full_mem_entry::from_bfd (bfd *cbfd, asection *osec, int *bfd_offset) +{ + uint32_t len; + uint64_t addr; + + /* Get len. */ + bfdcore_read (cbfd, osec, &len, sizeof (len), bfd_offset); + len = netorder32 (len); + + /* Get addr. */ + bfdcore_read (cbfd, osec, &addr, sizeof (addr), + bfd_offset); + addr = netorder64 (addr); + + record_full_mem_entry mem (addr, len); + + /* Get val. */ + bfdcore_read (cbfd, osec, mem.get_loc (), + len, bfd_offset); + + if (record_debug) + gdb_printf (gdb_stdlog, + " Reading memory %s (1 plus " + "%lu plus %lu plus %d bytes)\n", + paddress (get_current_arch (), + mem.addr), + (unsigned long) sizeof (addr), + (unsigned long) sizeof (len), + len); + + return mem; +} + +void +record_full_entry::from_bfd (bfd *cbfd, asection *osec, int *bfd_offset) +{ + uint8_t rectype; + record_full_entry rec; + bfdcore_read (cbfd, osec, &rectype, sizeof (rectype), bfd_offset); switch (rectype) { case record_full_reg: /* reg */ { - /* Get register number to regnum. */ - bfdcore_read (cbfd, osec, ®num, sizeof (regnum), bfd_offset); - regnum = netorder32 (regnum); - - record_full_entry rec (record_full_reg, cache->arch (), regnum); - - /* Get val. */ - bfdcore_read (cbfd, osec, rec.get_loc (), - rec.reg ().len, bfd_offset); - - if (record_debug) - gdb_printf (gdb_stdlog, - " Reading register %d (1 " - "plus %lu plus %d bytes)\n", - rec.reg ().num, - (unsigned long) sizeof (regnum), - rec.reg ().len); - - record_full_arch_list_add (rec); + rec.entry = std::move (record_full_reg_entry::from_bfd + (cbfd, osec, bfd_offset)); break; } case record_full_mem: /* mem */ { - /* Get len. */ - bfdcore_read (cbfd, osec, &len, sizeof (len), bfd_offset); - len = netorder32 (len); - - /* Get addr. */ - bfdcore_read (cbfd, osec, &addr, sizeof (addr), bfd_offset); - addr = netorder64 (addr); - - record_full_entry rec (record_full_mem, addr, len); - - /* Get val. */ - bfdcore_read (cbfd, osec, rec.get_loc (), len, bfd_offset); - - if (record_debug) - gdb_printf (gdb_stdlog, - " Reading memory %s (1 plus " - "%lu plus %lu plus %d bytes)\n", - paddress (get_current_arch (), - rec.mem ().addr), - (unsigned long) sizeof (addr), - (unsigned long) sizeof (len), - rec.mem ().len); - - record_full_arch_list_add (rec); + rec.entry = std::move (record_full_mem_entry::from_bfd + (cbfd, osec, bfd_offset)); break; } @@ -2307,6 +2363,39 @@ record_full_read_entry_from_bfd (bfd *cbfd, asection *osec, int *bfd_offset) bfd_get_filename (cbfd))); break; } + record_full_arch_list_add (rec); +} + +void +record_full_instruction::from_bfd (bfd *cbfd, asection *osec, int *bfd_offset) +{ + uint32_t eff_count = 0; + uint8_t sigval; + uint32_t insn_num; + + /* First read the generic information for an instruction. */ + bfdcore_read (cbfd, osec, &sigval, sizeof (uint8_t), bfd_offset); + bfdcore_read (cbfd, osec, &eff_count, sizeof (uint32_t), + bfd_offset); + bfdcore_read (cbfd, osec, &insn_num, sizeof (uint32_t), + bfd_offset); + + record_full_incomplete_instruction.insn_num = netorder32 (insn_num); + if (sigval != GDB_SIGNAL_0) + record_full_incomplete_instruction.sigval = (gdb_signal) sigval; + + record_full_incomplete_instruction.pc + = record_full_reg_entry::from_bfd (cbfd, osec, bfd_offset); + + eff_count = netorder32 (eff_count); + + /* This deals with all the side effects. */ + while (eff_count > 0) + { + eff_count--; + + record_full_entry::from_bfd (cbfd, osec, bfd_offset); + } } /* Restore the execution log from core file CBFD. */ @@ -2352,30 +2441,9 @@ record_full_restore (struct bfd &cbfd) { while (bfd_offset < osec_size) { - uint8_t sigval; - uint32_t eff_count, insn_num; - record_full_reset_incomplete (); - /* Frst read the generic information for an instruction. */ - bfdcore_read (&cbfd, osec, &sigval, sizeof (uint8_t), &bfd_offset); - bfdcore_read (&cbfd, osec, &eff_count, sizeof (uint32_t), - &bfd_offset); - bfdcore_read (&cbfd, osec, &insn_num, sizeof (uint32_t), - &bfd_offset); - - record_full_incomplete_instruction.insn_num = netorder32 (insn_num); - if (sigval != GDB_SIGNAL_0) - record_full_incomplete_instruction.sigval = (gdb_signal) sigval; - eff_count = netorder32 (eff_count); - - /* This deals with all the side effects. */ - while (eff_count > 0) - { - eff_count--; - - record_full_read_entry_from_bfd (&cbfd, osec, &bfd_offset); - } + record_full_instruction::from_bfd (&cbfd, osec, &bfd_offset); record_full_save_instruction (); } @@ -2431,67 +2499,104 @@ cmd_record_full_restore (const char *args, int from_tty) record_full_open (nullptr, from_tty); } -static void -record_full_write_entry_to_bfd (record_full_entry &entry, - const gdb_bfd_ref_ptr &obfd, - asection *osec, int *bfd_offset, - gdbarch *gdbarch) +void +record_full_reg_entry::to_bfd (const gdb_bfd_ref_ptr &obfd, + asection *osec, int *bfd_offset) +{ + uint32_t regnum; + + if (record_debug) + gdb_printf (gdb_stdlog, + " Writing register %d (1 " + "plus %lu plus %d bytes)\n", + this->num, + (unsigned long) sizeof (regnum), + this->len); + + /* Write regnum. */ + regnum = netorder32 (this->num); + bfdcore_write (obfd.get (), osec, ®num, sizeof (regnum), bfd_offset); + + /* Write regval. */ + bfdcore_write (obfd.get (), osec, this->get_loc (), this->len, bfd_offset); +} + +void +record_full_mem_entry::to_bfd (const gdb_bfd_ref_ptr &obfd, + asection *osec, int *bfd_offset, + gdbarch *gdbarch) +{ + uint32_t len; + uint64_t addr; + + if (record_debug) + gdb_printf (gdb_stdlog, + " Writing memory %s (1 plus " + "%lu plus %lu plus %d bytes)\n", + paddress (gdbarch, this->addr), + (unsigned long) sizeof (addr), + (unsigned long) sizeof (len), + this->len); + + /* Write memlen. */ + len = netorder32 (this->len); + bfdcore_write (obfd.get (), osec, &len, sizeof (len), bfd_offset); + + /* Write memaddr. */ + addr = netorder64 (this->addr); + bfdcore_write (obfd.get (), osec, &addr, sizeof (addr), bfd_offset); + + /* Write memval. */ + bfdcore_write (obfd.get (), osec, this->get_loc (), this->len, bfd_offset); +} + +void +record_full_entry::to_bfd (const gdb_bfd_ref_ptr &obfd, asection *osec, + int *bfd_offset, gdbarch *gdbarch) { /* Save entry. */ uint8_t type; - uint32_t regnum, len; - uint64_t addr; - type = entry.type (); + type = this->type (); bfdcore_write (obfd.get (), osec, &type, sizeof (type), bfd_offset); switch (type) { case record_full_reg: /* reg */ - { - auto ® = entry.reg (); - if (record_debug) - gdb_printf (gdb_stdlog, - " Writing register %d (1 " - "plus %lu plus %d bytes)\n", - reg.num, - (unsigned long) sizeof (regnum), - reg.len); - - /* Write regnum. */ - regnum = netorder32 (reg.num); - bfdcore_write (obfd.get (), osec, ®num, sizeof (regnum), bfd_offset); - - /* Write regval. */ - bfdcore_write (obfd.get (), osec, entry.get_loc (), reg.len, bfd_offset); - break; - } + reg ().to_bfd (obfd, osec, bfd_offset); + break; case record_full_mem: /* mem */ - { - auto &mem = entry.mem (); - if (record_debug) - gdb_printf (gdb_stdlog, - " Writing memory %s (1 plus " - "%lu plus %lu plus %d bytes)\n", - paddress (gdbarch, mem.addr), - (unsigned long) sizeof (addr), - (unsigned long) sizeof (len), - mem.len); - - /* Write memlen. */ - len = netorder32 (mem.len); - bfdcore_write (obfd.get (), osec, &len, sizeof (len), bfd_offset); - - /* Write memaddr. */ - addr = netorder64 (mem.addr); - bfdcore_write (obfd.get (), osec, &addr, sizeof (addr), bfd_offset); - - /* Write memval. */ - bfdcore_write (obfd.get (), osec, entry.get_loc (), mem.len, - bfd_offset); - break; - } + mem ().to_bfd (obfd, osec, bfd_offset, gdbarch); + break; + } +} + +void +record_full_instruction::to_bfd (gdb_bfd_ref_ptr obfd, asection *osec, + int *bfd_offset, gdbarch *gdbarch) +{ + uint32_t eff_count = (uint32_t) this->effects.size (); + uint32_t insn_num = this->insn_num; + uint8_t sigval = (this->sigval.has_value ()) + ? this->sigval.value () + : GDB_SIGNAL_0; + + /* Signal. */ + bfdcore_write (obfd.get (), osec, &sigval, sizeof (sigval), bfd_offset); + eff_count = netorder32 (eff_count); + /* Number of effects. */ + bfdcore_write (obfd.get (), osec, &eff_count, sizeof (eff_count), + bfd_offset); + /* Instruction number. */ + bfdcore_write (obfd.get (), osec, &insn_num, sizeof (uint32_t), + bfd_offset); + + this->pc.to_bfd (obfd, osec, bfd_offset); + + for (auto &entry : this->effects) + { + entry.to_bfd (obfd, osec, bfd_offset, gdbarch); } } @@ -2536,6 +2641,7 @@ record_full_base_target::save_record (const char *recfilename) { /* Number of effects of an instruction. */ save_size += sizeof (uint32_t) + sizeof (uint8_t) + sizeof (uint32_t); + save_size += 4 + record_full_list[i].pc.len; for (auto &entry : record_full_list[i].effects) switch (entry.type ()) { @@ -2577,28 +2683,7 @@ record_full_base_target::save_record (const char *recfilename) record list. */ for (int i = 0; i < record_full_list.size (); i++) { - uint32_t eff_count = (uint32_t) record_full_list[i].effects.size (); - uint32_t insn_num = record_full_list[i].insn_num; - uint8_t sigval = (record_full_list[i].sigval.has_value ()) - ? record_full_list[i].sigval.value () - : GDB_SIGNAL_0; - - /* Signal. */ - bfdcore_write (obfd.get (), osec, &sigval, sizeof (sigval), &bfd_offset); - /* Number of effects. */ - eff_count = netorder32 (eff_count); - bfdcore_write (obfd.get (), osec, &eff_count, sizeof (eff_count), - &bfd_offset); - /* Instruction number. */ - bfdcore_write (obfd.get (), osec, &insn_num, sizeof (insn_num), - &bfd_offset); - - for (auto &entry : record_full_list[i].effects) - { - record_full_write_entry_to_bfd (entry, obfd, osec, &bfd_offset, - gdbarch); - } - + record_full_list[i].to_bfd (obfd, osec, &bfd_offset, gdbarch); /* Execute entry. */ if (i < record_full_next_insn) record_full_list[i].exec_insn (regcache); @@ -2675,6 +2760,9 @@ maintenance_print_record_instruction (const char *args, int from_tty) auto to_print = record_full_list.begin () + offset; gdbarch *arch = current_inferior ()->arch (); + struct value_print_options opts; + get_user_print_options (&opts); + opts.raw = true; for (auto &entry : to_print->effects) { @@ -2688,9 +2776,6 @@ maintenance_print_record_instruction (const char *args, int from_tty) entry.get_loc ()); gdb_printf ("Register %s changed: ", gdbarch_register_name (arch, entry.reg ().num)); - struct value_print_options opts; - get_user_print_options (&opts); - opts.raw = true; value_print (val, gdb_stdout, &opts); gdb_printf ("\n"); break; @@ -2709,6 +2794,12 @@ maintenance_print_record_instruction (const char *args, int from_tty) } } } + type *regtype = gdbarch_register_type (arch, to_print->pc.num); + value *val = value_from_contents (regtype, to_print->pc.get_loc ()); + gdb_printf ("Register %s changed: ", + gdbarch_register_name (arch, to_print->pc.num)); + value_print (val, gdb_stdout, &opts); + gdb_printf ("\n"); } INIT_GDB_FILE (record_full) -- 2.54.0