From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lndn.lancelotsix.com (vps-42846194.vps.ovh.net [IPv6:2001:41d0:801:2000::2400]) by sourceware.org (Postfix) with ESMTPS id AD858395B476 for ; Tue, 6 Dec 2022 14:52:13 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org AD858395B476 Authentication-Results: sourceware.org; dmarc=pass (p=reject dis=none) header.from=lancelotsix.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=lancelotsix.com Received: from octopus (unknown [IPv6:2a02:390:9086:0:edc6:ce5e:38d6:84f9]) by lndn.lancelotsix.com (Postfix) with ESMTPSA id ED88F806B1; Tue, 6 Dec 2022 14:52:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=lancelotsix.com; s=2021; t=1670338332; bh=ZS5Qim7lO+b5zCkXL21Bt7QbhOKEioXJSRAj3Hs2kgY=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=K/BiFFJa0MkupQXkEe1c8IpkDsukLGcG6ksfsnhdJfPTSfWZpCPaG5FsiK79jOCkq 8trsyIP1JwyZwWbkxvcnAVAJ7l4Qg3rbYxmoROTfTmNnW9qBRO8uyeyBrtwiC8+k9t DQvhp+1bNN1ir1STwtnCgiIzgWCfhyYPHp2cScUFVac/CQdmRu/usG8BXNfshBOqsN phj6mJ/Gnie+0LzMPGKAw6QzEyfyLkLiN7buEOZ+3gMVDy9tFQsvWEB89m+WAl6iyY PXFFk3hM772WSJmdJYeQz8HmidCmgo/PDJtesTsAVWR10dm7uh8IyuWQjBXEyD4KZz 1sChnKXlKgOAA== Date: Tue, 6 Dec 2022 14:52:02 +0000 From: Lancelot SIX To: Simon Marchi Cc: gdb-patches@sourceware.org, Simon Marchi Subject: Re: [pushed] gdb: make frame_register static Message-ID: <20221206145202.lah7uwybcvv4akqi@octopus> References: <20221201183249.1062823-1-simon.marchi@efficios.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20221201183249.1062823-1-simon.marchi@efficios.com> X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.5.11 (lndn.lancelotsix.com [0.0.0.0]); Tue, 06 Dec 2022 14:52:12 +0000 (UTC) X-Spam-Status: No, score=-10.3 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,GIT_PATCH_0,RCVD_IN_SBL_CSS,SPF_HELO_NONE,SPF_PASS,TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: Hi Simon, The function frame_register is used outside of frame.c in the rocgdb downstream port. Do you think we could revert this patch here? Otherwise, this can be a change we keep downstream. Best, Lancelot. On Thu, Dec 01, 2022 at 01:32:49PM -0500, Simon Marchi via Gdb-patches wrote: > From: Simon Marchi > > It is only used inside frame.c. > > Change-Id: I44eb46a5992412f8f8b4954b2284b0ef3b549504 > --- > gdb/frame.c | 7 ++++++- > gdb/frame.h | 11 ----------- > 2 files changed, 6 insertions(+), 12 deletions(-) > > diff --git a/gdb/frame.c b/gdb/frame.c > index c8c4ec817a42..b4079ef833e8 100644 > --- a/gdb/frame.c > +++ b/gdb/frame.c > @@ -1166,7 +1166,12 @@ frame_register_unwind (frame_info_ptr next_frame, int regnum, > release_value (value); > } > > -void > +/* Get the value of the register that belongs to this FRAME. This > + function is a wrapper to the call sequence ``frame_register_unwind > + (get_next_frame (FRAME))''. As per frame_register_unwind(), if > + VALUEP is NULL, the registers value is not fetched/computed. */ > + > +static void > frame_register (frame_info_ptr frame, int regnum, > int *optimizedp, int *unavailablep, enum lval_type *lvalp, > CORE_ADDR *addrp, int *realnump, gdb_byte *bufferp) > diff --git a/gdb/frame.h b/gdb/frame.h > index 6b841fd50636..100ab1fe48eb 100644 > --- a/gdb/frame.h > +++ b/gdb/frame.h > @@ -511,17 +511,6 @@ extern ULONGEST get_frame_register_unsigned (frame_info_ptr frame, > extern bool read_frame_register_unsigned (frame_info_ptr frame, > int regnum, ULONGEST *val); > > -/* Get the value of the register that belongs to this FRAME. This > - function is a wrapper to the call sequence ``frame_register_unwind > - (get_next_frame (FRAME))''. As per frame_register_unwind(), if > - VALUEP is NULL, the registers value is not fetched/computed. */ > - > -extern void frame_register (frame_info_ptr frame, int regnum, > - int *optimizedp, int *unavailablep, > - enum lval_type *lvalp, > - CORE_ADDR *addrp, int *realnump, > - gdb_byte *valuep); > - > /* The reverse. Store a register value relative to the specified > frame. Note: this call makes the frame's state undefined. The > register and frame caches must be flushed. */ > -- > 2.38.1 >