From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from eggs.gnu.org (eggs.gnu.org [IPv6:2001:470:142:3::10]) by sourceware.org (Postfix) with ESMTPS id E50463858D28 for ; Tue, 20 Sep 2022 12:47:17 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org E50463858D28 Received: from fencepost.gnu.org ([2001:470:142:3::e]:40452) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1oaceh-0004pb-Ln; Tue, 20 Sep 2022 08:47:16 -0400 Received: from [87.69.77.57] (port=3382 helo=home-c4e4a596f7) by fencepost.gnu.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1oaceZ-0000fH-QO; Tue, 20 Sep 2022 08:47:15 -0400 Date: Tue, 20 Sep 2022 15:47:14 +0300 Message-Id: <83pmfqw6ul.fsf@gnu.org> From: Eli Zaretskii To: Luis Machado Cc: gdb-patches@sourceware.org In-Reply-To: <20220920123012.189293-1-luis.machado@arm.com> (message from Luis Machado via Gdb-patches on Tue, 20 Sep 2022 13:30:12 +0100) Subject: Re: [PATCH] [Arm] Remove dead FPA code References: <20220920123012.189293-1-luis.machado@arm.com> X-Spam-Status: No, score=1.9 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, RCVD_IN_BARRACUDACENTRAL, SPF_HELO_PASS, SPF_PASS, TXREP autolearn=no autolearn_force=no version=3.4.6 X-Spam-Level: * 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.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 20 Sep 2022 12:47:19 -0000 > Date: Tue, 20 Sep 2022 13:30:12 +0100 > From: Luis Machado via Gdb-patches > > FPA has been deprecated for a while now. It was removed from GCC in 2012, > but somehow it managed to survive in GDB for a little while longer. > > It is time to let FPA rest in peace. > > While at it, make VFP the default floating point model for 32-bit Arm. > > It would be nice to get some *BSD testing as some changes touch that code > and I'm not properly setup to validate the changes for those OS'. > --- > gdb/NEWS | 4 + > gdb/aarch64-linux-nat.c | 4 +- > gdb/arch/arm.c | 5 - > gdb/arch/arm.h | 9 - > gdb/arm-bsd-tdep.c | 48 ---- > gdb/arm-linux-nat.c | 106 +------- > gdb/arm-linux-tdep.c | 131 +-------- > gdb/arm-linux-tdep.h | 29 -- > gdb/arm-netbsd-nat.c | 4 +- > gdb/arm-tdep.c | 286 +------------------- > gdb/arm-tdep.h | 3 - > gdb/doc/gdb.texinfo | 13 +- > gdb/features/Makefile | 2 - > gdb/features/arm/arm-fpa.c | 23 -- > gdb/features/arm/arm-fpa.xml | 23 -- > gdb/features/arm/arm-m-profile-with-fpa.c | 39 --- > gdb/features/arm/arm-m-profile-with-fpa.xml | 39 --- > gdb/target-descriptions.c | 4 - > gdb/testsuite/gdb.base/long_long.exp | 18 +- > gdbserver/linux-arm-low.cc | 7 +- > gdbsupport/tdesc.cc | 1 - > gdbsupport/tdesc.h | 1 - > 22 files changed, 33 insertions(+), 766 deletions(-) > delete mode 100644 gdb/features/arm/arm-fpa.c > delete mode 100644 gdb/features/arm/arm-fpa.xml > delete mode 100644 gdb/features/arm/arm-m-profile-with-fpa.c > delete mode 100644 gdb/features/arm/arm-m-profile-with-fpa.xml OK for the NEWS part, thanks.