From patchwork Thu Dec 21 19:16:26 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Simon Marchi X-Patchwork-Id: 82694 Return-Path: X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 9EBA938618BF for ; Thu, 21 Dec 2023 19:17:57 +0000 (GMT) X-Original-To: gdb-patches@sourceware.org Delivered-To: gdb-patches@sourceware.org Received: from simark.ca (simark.ca [158.69.221.121]) by sourceware.org (Postfix) with ESMTPS id 78531385DC02 for ; Thu, 21 Dec 2023 19:17:20 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 78531385DC02 Authentication-Results: sourceware.org; dmarc=fail (p=none dis=none) header.from=efficios.com Authentication-Results: sourceware.org; spf=fail smtp.mailfrom=efficios.com ARC-Filter: OpenARC Filter v1.0.0 sourceware.org 78531385DC02 Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=158.69.221.121 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1703186241; cv=none; b=YzWmJaofennxcP+5gOfsuLUAa7yKMvGEke0YlyoCOTr0puLD6+2NQacMEmsPfc2aDLwTZcBvTyWmOLWp2O/Un5EILbpA0yWnk9fuRtSuOI/0QlB8EgjesL5fFOpBKBBz7LzM8XpPIjRF042ub+uQg5JrZoLT5GT/RIpiO173V1Y= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1703186241; c=relaxed/simple; bh=veejhO1ahb7WHZ+ijpdq/ovqyWcRbNJZ7ihDjEo7azg=; h=From:To:Subject:Date:Message-ID:MIME-Version; b=DwGFs+tmYhu4/LtObdN4AOUKpOQBm7rsUFYMlCHYER26Kdkk4JLfntJ3ve0mIuubgjuZNr8/BExTZlgUOoD2tRuTcFAe6/CsjiBoT+hP/PU+S3he/mZeoZBLdigsFKsdygR4m/ea0VuB1X6e5QEjeeX2LS9dXx675O9tHf66eLA= ARC-Authentication-Results: i=1; server2.sourceware.org Received: from smarchi-efficios.internal.efficios.com (192-222-143-198.qc.cable.ebox.net [192.222.143.198]) (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 ESMTPSA id 84B061E11E; Thu, 21 Dec 2023 14:17:19 -0500 (EST) From: Simon Marchi To: gdb-patches@sourceware.org Cc: Simon Marchi Subject: [PATCH 5/9] gdb: remove read_frame_register_value's frame parameter Date: Thu, 21 Dec 2023 14:16:26 -0500 Message-ID: <20231221191716.257256-6-simon.marchi@efficios.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20231221191716.257256-1-simon.marchi@efficios.com> References: <20231221191716.257256-1-simon.marchi@efficios.com> MIME-Version: 1.0 X-Spam-Status: No, score=-3496.6 required=5.0 tests=BAYES_00, GIT_PATCH_0, KAM_DMARC_NONE, KAM_DMARC_STATUS, SPF_HELO_PASS, SPF_SOFTFAIL, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org 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+patchwork=sourceware.org@sourceware.org By now, all register struct values should have a valid next frame id (assuming they are created using value::allocate_register or value::allocate_register_lazy), so there should be no need to pass a frame alongside the value to read_frame_register_value. Remove the frame parameter and adjust read_frame_register_value accordingly. While at it, make read_frame_register_value static, it's only used in findvar.c. Change-Id: I118959ef8c628499297c67810916e8ba9934bfac --- gdb/findvar.c | 23 +++++++++++++---------- gdb/value.h | 3 --- 2 files changed, 13 insertions(+), 13 deletions(-) diff --git a/gdb/findvar.c b/gdb/findvar.c index 7c6c837ae12e..838d850e821d 100644 --- a/gdb/findvar.c +++ b/gdb/findvar.c @@ -767,24 +767,27 @@ default_value_from_register (gdbarch *gdbarch, type *type, int regnum, } /* VALUE must be an lval_register value. If regnum is the value's - associated register number, and len the length of the values type, - read one or more registers in FRAME, starting with register REGNUM, + associated register number, and len the length of the value's type, + read one or more registers in VALUE's frame, starting with register REGNUM, until we've read LEN bytes. If any of the registers we try to read are optimized out, then mark the complete resulting value as optimized out. */ -void -read_frame_register_value (struct value *value, frame_info_ptr frame) +static void +read_frame_register_value (value *value) { - struct gdbarch *gdbarch = get_frame_arch (frame); + gdb_assert (value->lval () == lval_register); + + frame_info_ptr next_frame = frame_find_by_id (VALUE_NEXT_FRAME_ID (value)); + gdb_assert (next_frame != nullptr); + + gdbarch *gdbarch = frame_unwind_arch (next_frame); LONGEST offset = 0; LONGEST reg_offset = value->offset (); int regnum = VALUE_REGNUM (value); int len = type_length_units (check_typedef (value->type ())); - gdb_assert (value->lval () == lval_register); - /* Skip registers wholly inside of REG_OFFSET. */ while (reg_offset >= register_size (gdbarch, regnum)) { @@ -795,7 +798,7 @@ read_frame_register_value (struct value *value, frame_info_ptr frame) /* Copy the data. */ while (len > 0) { - struct value *regval = get_frame_register_value (frame, regnum); + struct value *regval = frame_unwind_register_value (next_frame, regnum); int reg_len = type_length_units (regval->type ()) - reg_offset; /* If the register length is larger than the number of bytes @@ -852,7 +855,7 @@ value_from_register (struct type *type, int regnum, frame_info_ptr frame) v = gdbarch_value_from_register (gdbarch, type, regnum, frame); /* Get the data. */ - read_frame_register_value (v, frame); + read_frame_register_value (v); } return v; @@ -895,7 +898,7 @@ address_from_register (int regnum, frame_info_ptr frame) } value *value = gdbarch_value_from_register (gdbarch, type, regnum, frame); - read_frame_register_value (value, frame); + read_frame_register_value (value); if (value->optimized_out ()) { diff --git a/gdb/value.h b/gdb/value.h index 78abcac7e739..9d7630ef07b3 100644 --- a/gdb/value.h +++ b/gdb/value.h @@ -1119,9 +1119,6 @@ extern value *default_value_from_register (gdbarch *gdbarch, type *type, int regnum, frame_info_ptr this_frame); -extern void read_frame_register_value (struct value *value, - frame_info_ptr frame); - extern struct value *value_from_register (struct type *type, int regnum, frame_info_ptr frame);