From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 2156) id 069DF385734A; Thu, 29 Sep 2022 17:07:59 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 069DF385734A DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1664471279; bh=472RIos9+89bTx6xj0LUWZFD4FGzqlHc7khC+yfInUI=; h=From:To:Subject:Date:From; b=g64kRepP/RhWj1lqqZma3UzUnzYb7EaROYtQ124qkYc+YQjSqtij03VeZxCjWzIkd g19adh/mRfPoKOqWpsvUbySllff/gBXwbL2tMNFUdFic0V4cVXtp8zn7fuWQsm5S4X bYgwW0m/lxi9JaEkLRBWddqLBcn+REuEzcDAJeGE= Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: Keith Seitz To: gdb-cvs@sourceware.org Subject: [binutils-gdb] cleanup: Add missing feature/ XML files to Makefile X-Act-Checkin: binutils-gdb X-Git-Author: Keith Seitz X-Git-Refname: refs/heads/master X-Git-Oldrev: 55b0ce4403e29e5c5295bc26024281c62e10ca3c X-Git-Newrev: ee8cc2b3f2d121718165b1cab2f537bbf30cae89 Message-Id: <20220929170759.069DF385734A@sourceware.org> Date: Thu, 29 Sep 2022 17:07:59 +0000 (GMT) List-Id: https://sourceware.org/git/gitweb.cgi?p=3Dbinutils-gdb.git;h=3Dee8cc2b3f2d1= 21718165b1cab2f537bbf30cae89 commit ee8cc2b3f2d121718165b1cab2f537bbf30cae89 Author: Keith Seitz Date: Mon Sep 12 07:56:58 2022 -0700 cleanup: Add missing feature/ XML files to Makefile =20 This patch adds some missing .xml files to features/Makefile so that wh= en the directory's C files are regenerated, all files are appropriately remade. =20 This has demonstrated that there have been several "misses" in regenera= ting files in this directory. Namely, arm-secext.c and sparc{32,64}-solaris.= c. For the former case, there was what essentially amounts to a typo regarding= the create feature function's name. In the later case, this file has missed= at least one important update in July, 2020, when allocate_target_description was changed to return a unique pointer. =20 Those corrections are included. Diff: --- gdb/features/Makefile | 4 ++++ gdb/features/arm/arm-secext.c | 2 +- gdb/features/sparc/sparc32-solaris.c | 16 ++++++++-------- gdb/features/sparc/sparc64-solaris.c | 22 +++++++++++----------- 4 files changed, 24 insertions(+), 20 deletions(-) diff --git a/gdb/features/Makefile b/gdb/features/Makefile index 4bc85962dfe..1f2b8963ad8 100644 --- a/gdb/features/Makefile +++ b/gdb/features/Makefile @@ -108,6 +108,7 @@ XMLTOC =3D \ nds32.xml \ nios2.xml \ or1k.xml \ + or1k-linux.xml \ rs6000/powerpc-32.xml \ rs6000/powerpc-32l.xml \ rs6000/powerpc-403.xml \ @@ -164,6 +165,8 @@ XMLTOC =3D \ s390x-vx-linux64.xml \ s390-gs-linux64.xml \ s390x-gs-linux64.xml \ + sparc/sparc32-solaris.xml \ + sparc/sparc64-solaris.xml \ z80.xml =20 TDESC_CFILES =3D $(patsubst %.xml,%.c,$(XMLTOC)) @@ -209,6 +212,7 @@ FEATURE_XMLFILES =3D aarch64-core.xml \ arm/arm-m-profile-mve.xml \ arm/arm-m-system.xml \ arm/arm-m-profile-with-fpa.xml \ + arm/arm-secext.xml \ arm/arm-tls.xml \ arm/arm-vfpv2.xml \ arm/arm-vfpv3.xml \ diff --git a/gdb/features/arm/arm-secext.c b/gdb/features/arm/arm-secext.c index 39ef4afb05f..c71cffd2db6 100644 --- a/gdb/features/arm/arm-secext.c +++ b/gdb/features/arm/arm-secext.c @@ -4,7 +4,7 @@ #include "gdbsupport/tdesc.h" =20 static int -create_feature_arm_arm_m_system (struct target_desc *result, long regnum) +create_feature_arm_arm_secext (struct target_desc *result, long regnum) { struct tdesc_feature *feature; =20 diff --git a/gdb/features/sparc/sparc32-solaris.c b/gdb/features/sparc/spar= c32-solaris.c index 2f13292b72b..c2f16062fc1 100644 --- a/gdb/features/sparc/sparc32-solaris.c +++ b/gdb/features/sparc/sparc32-solaris.c @@ -9,14 +9,14 @@ struct target_desc *tdesc_sparc32_solaris; static void initialize_tdesc_sparc32_solaris (void) { - struct target_desc *result =3D allocate_target_description (); - struct tdesc_feature *feature; + target_desc_up result =3D allocate_target_description (); + set_tdesc_architecture (result.get (), bfd_scan_arch ("sparc")); =20 - set_tdesc_architecture (result, bfd_scan_arch ("sparc")); + set_tdesc_osabi (result.get (), osabi_from_tdesc_string ("Solaris")); =20 - set_tdesc_osabi (result, osabi_from_tdesc_string ("Solaris")); + struct tdesc_feature *feature; =20 - feature =3D tdesc_create_feature (result, "org.gnu.gdb.sparc.cpu"); + feature =3D tdesc_create_feature (result.get (), "org.gnu.gdb.sparc.cpu"= ); tdesc_create_reg (feature, "g0", 0, 1, NULL, 32, "uint32"); tdesc_create_reg (feature, "g1", 1, 1, NULL, 32, "uint32"); tdesc_create_reg (feature, "g2", 2, 1, NULL, 32, "uint32"); @@ -50,7 +50,7 @@ initialize_tdesc_sparc32_solaris (void) tdesc_create_reg (feature, "fp", 30, 1, NULL, 32, "uint32"); tdesc_create_reg (feature, "i7", 31, 1, NULL, 32, "uint32"); =20 - feature =3D tdesc_create_feature (result, "org.gnu.gdb.sparc.cp0"); + feature =3D tdesc_create_feature (result.get (), "org.gnu.gdb.sparc.cp0"= ); tdesc_create_reg (feature, "y", 64, 1, NULL, 32, "uint32"); tdesc_create_reg (feature, "psr", 65, 1, NULL, 32, "uint32"); tdesc_create_reg (feature, "wim", 66, 1, NULL, 32, "uint32"); @@ -60,7 +60,7 @@ initialize_tdesc_sparc32_solaris (void) tdesc_create_reg (feature, "fsr", 70, 1, NULL, 32, "uint32"); tdesc_create_reg (feature, "csr", 71, 1, NULL, 32, "uint32"); =20 - feature =3D tdesc_create_feature (result, "org.gnu.gdb.sparc.fpu"); + feature =3D tdesc_create_feature (result.get (), "org.gnu.gdb.sparc.fpu"= ); tdesc_create_reg (feature, "f0", 32, 1, NULL, 32, "ieee_single"); tdesc_create_reg (feature, "f1", 33, 1, NULL, 32, "ieee_single"); tdesc_create_reg (feature, "f2", 34, 1, NULL, 32, "ieee_single"); @@ -94,5 +94,5 @@ initialize_tdesc_sparc32_solaris (void) tdesc_create_reg (feature, "f30", 62, 1, NULL, 32, "ieee_single"); tdesc_create_reg (feature, "f31", 63, 1, NULL, 32, "ieee_single"); =20 - tdesc_sparc_solaris =3D result; + tdesc_sparc32_solaris =3D result.release (); } diff --git a/gdb/features/sparc/sparc64-solaris.c b/gdb/features/sparc/spar= c64-solaris.c index ecc7b0fa58f..bb8f6853841 100644 --- a/gdb/features/sparc/sparc64-solaris.c +++ b/gdb/features/sparc/sparc64-solaris.c @@ -9,14 +9,14 @@ struct target_desc *tdesc_sparc64_solaris; static void initialize_tdesc_sparc64_solaris (void) { - struct target_desc *result =3D allocate_target_description (); - struct tdesc_feature *feature; + target_desc_up result =3D allocate_target_description (); + set_tdesc_architecture (result.get (), bfd_scan_arch ("sparc:v9")); =20 - set_tdesc_architecture (result, bfd_scan_arch ("sparc")); + set_tdesc_osabi (result.get (), osabi_from_tdesc_string ("Solaris")); =20 - set_tdesc_osabi (result, osabi_from_tdesc_string ("Solaris")); + struct tdesc_feature *feature; =20 - feature =3D tdesc_create_feature (result, "org.gnu.gdb.sparc.cpu"); + feature =3D tdesc_create_feature (result.get (), "org.gnu.gdb.sparc.cpu"= ); tdesc_create_reg (feature, "g0", 0, 1, NULL, 64, "uint64"); tdesc_create_reg (feature, "g1", 1, 1, NULL, 64, "uint64"); tdesc_create_reg (feature, "g2", 2, 1, NULL, 64, "uint64"); @@ -47,18 +47,18 @@ initialize_tdesc_sparc64_solaris (void) tdesc_create_reg (feature, "i3", 27, 1, NULL, 64, "uint64"); tdesc_create_reg (feature, "i4", 28, 1, NULL, 64, "uint64"); tdesc_create_reg (feature, "i5", 29, 1, NULL, 64, "uint64"); - tdesc_create_reg (feature, "fp", 30, 1, NULL, 32, "uint64"); + tdesc_create_reg (feature, "fp", 30, 1, NULL, 64, "uint64"); tdesc_create_reg (feature, "i7", 31, 1, NULL, 64, "uint64"); =20 - feature =3D tdesc_create_feature (result, "org.gnu.gdb.sparc.cp0"); + feature =3D tdesc_create_feature (result.get (), "org.gnu.gdb.sparc.cp0"= ); tdesc_create_reg (feature, "pc", 80, 1, NULL, 64, "code_ptr"); tdesc_create_reg (feature, "npc", 81, 1, NULL, 64, "code_ptr"); tdesc_create_reg (feature, "state", 82, 1, NULL, 64, "uint64"); - tdesc_create_reg (feature, "fsr", 83, 1, NULL, 32, "ieee_single"); - tdesc_create_reg (feature, "fprs", 84, 1, NULL, 32, "ieee_single"); + tdesc_create_reg (feature, "fsr", 83, 1, NULL, 64, "uint64"); + tdesc_create_reg (feature, "fprs", 84, 1, NULL, 64, "uint64"); tdesc_create_reg (feature, "y", 85, 1, NULL, 64, "uint64"); =20 - feature =3D tdesc_create_feature (result, "org.gnu.gdb.sparc.fpu"); + feature =3D tdesc_create_feature (result.get (), "org.gnu.gdb.sparc.fpu"= ); tdesc_create_reg (feature, "f0", 32, 1, NULL, 32, "ieee_single"); tdesc_create_reg (feature, "f1", 33, 1, NULL, 32, "ieee_single"); tdesc_create_reg (feature, "f2", 34, 1, NULL, 32, "ieee_single"); @@ -108,5 +108,5 @@ initialize_tdesc_sparc64_solaris (void) tdesc_create_reg (feature, "f60", 78, 1, NULL, 64, "ieee_double"); tdesc_create_reg (feature, "f62", 79, 1, NULL, 64, "ieee_double"); =20 - tdesc_sparc64_solaris =3D result; + tdesc_sparc64_solaris =3D result.release (); }