This is the mail archive of the
gdb-cvs@sourceware.org
mailing list for the GDB project.
[binutils-gdb] Make target_desc::compatible an std::vector
- From: Simon Marchi <simark at sourceware dot org>
- To: gdb-cvs at sourceware dot org
- Date: 5 Dec 2017 21:31:38 -0000
- Subject: [binutils-gdb] Make target_desc::compatible an std::vector
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=40e2a9835f6cf47b630babef3fd788e413ef2dc5
commit 40e2a9835f6cf47b630babef3fd788e413ef2dc5
Author: Simon Marchi <simon.marchi@polymtl.ca>
Date: Tue Dec 5 16:30:22 2017 -0500
Make target_desc::compatible an std::vector
This patch changes target_desc::compatible to be a vector of
bfd_arch_info *. This way, we don't need to manually free the vector in
the target_desc destructor.
gdb/ChangeLog:
* target-descriptions.c (arch_p): Remove typedef.
(DEF_VEC_P (arch_p)): Remove.
(struct target_desc) <compatible>: Change type to std::vector.
<~target_desc>: Don't manually free compatible.
(tdesc_compatible_p): Adjust.
(tdesc_add_compatible): Adjust.
(class print_c_tdesc) <visit_pre>: Adjust.
Diff:
---
gdb/ChangeLog | 10 ++++++++++
gdb/target-descriptions.c | 39 +++++++++------------------------------
2 files changed, 19 insertions(+), 30 deletions(-)
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 3abf5d2..2f38e1a 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,15 @@
2017-12-05 Simon Marchi <simon.marchi@polymtl.ca>
+ * target-descriptions.c (arch_p): Remove typedef.
+ (DEF_VEC_P (arch_p)): Remove.
+ (struct target_desc) <compatible>: Change type to std::vector.
+ <~target_desc>: Don't manually free compatible.
+ (tdesc_compatible_p): Adjust.
+ (tdesc_add_compatible): Adjust.
+ (class print_c_tdesc) <visit_pre>: Adjust.
+
+2017-12-05 Simon Marchi <simon.marchi@polymtl.ca>
+
* target-descriptions.c (property_s): Remove typedef.
(DEF_VEC_O (property_s)): Remove.
(struct target_desc) <properties>: Make an std::vector.
diff --git a/gdb/target-descriptions.c b/gdb/target-descriptions.c
index 940c2f5..75f835a 100644
--- a/gdb/target-descriptions.c
+++ b/gdb/target-descriptions.c
@@ -386,10 +386,6 @@ typedef struct tdesc_feature : tdesc_element
} *tdesc_feature_p;
DEF_VEC_P(tdesc_feature_p);
-/* A compatible architecture from a target description. */
-typedef const struct bfd_arch_info *arch_p;
-DEF_VEC_P(arch_p);
-
/* A target description. */
struct target_desc : tdesc_element
@@ -407,8 +403,6 @@ struct target_desc : tdesc_element
ix++)
delete feature;
VEC_free (tdesc_feature_p, features);
-
- VEC_free (arch_p, compatible);
}
target_desc (const target_desc &) = delete;
@@ -422,7 +416,7 @@ struct target_desc : tdesc_element
enum gdb_osabi osabi = GDB_OSABI_UNKNOWN;
/* The list of compatible architectures reported by the target. */
- VEC(arch_p) *compatible = NULL;
+ std::vector<const bfd_arch_info *> compatible;
/* Any architecture-specific properties specified by the target. */
std::vector<property> properties;
@@ -695,11 +689,7 @@ int
tdesc_compatible_p (const struct target_desc *target_desc,
const struct bfd_arch_info *arch)
{
- const struct bfd_arch_info *compat;
- int ix;
-
- for (ix = 0; VEC_iterate (arch_p, target_desc->compatible, ix, compat);
- ix++)
+ for (const bfd_arch_info *compat : target_desc->compatible)
{
if (compat == arch
|| arch->compatible (arch, compat)
@@ -1765,24 +1755,20 @@ void
tdesc_add_compatible (struct target_desc *target_desc,
const struct bfd_arch_info *compatible)
{
- const struct bfd_arch_info *compat;
- int ix;
-
/* If this instance of GDB is compiled without BFD support for the
compatible architecture, simply ignore it -- we would not be able
to handle it anyway. */
if (compatible == NULL)
return;
- for (ix = 0; VEC_iterate (arch_p, target_desc->compatible, ix, compat);
- ix++)
+ for (const bfd_arch_info *compat : target_desc->compatible)
if (compat == compatible)
internal_error (__FILE__, __LINE__,
_("Attempted to add duplicate "
"compatible architecture \"%s\""),
compatible->printable_name);
- VEC_safe_push (arch_p, target_desc->compatible, compatible);
+ target_desc->compatible.push_back (compatible);
}
void
@@ -1953,19 +1939,12 @@ public:
printf_unfiltered ("\n");
}
- int ix;
- const struct bfd_arch_info *compatible;
- struct property *prop;
-
- for (ix = 0; VEC_iterate (arch_p, e->compatible, ix, compatible);
- ix++)
- {
- printf_unfiltered
- (" tdesc_add_compatible (result, bfd_scan_arch (\"%s\"));\n",
- compatible->printable_name);
- }
+ for (const struct bfd_arch_info *compatible : e->compatible)
+ printf_unfiltered
+ (" tdesc_add_compatible (result, bfd_scan_arch (\"%s\"));\n",
+ compatible->printable_name);
- if (ix)
+ if (!e->compatible.empty ())
printf_unfiltered ("\n");
for (const property &prop : e->properties)