From 7aa913136675f4b81cd3a548e44bbdab6185abed Mon Sep 17 00:00:00 2001 From: Simon Marchi Date: Thu, 7 May 2020 11:08:54 -0400 Subject: [PATCH] gdb: make remove_dyn_prop a method of struct type Move remove_dyn_prop, currently a free function, to be a method of struct type. gdb/ChangeLog: * gdbtypes.h (struct type) : New method. (remove_dyn_prop): Remove. Update all users to use type::remove_dyn_prop. * gdbtypes.c (remove_dyn_prop): Rename to... (type::remove_dyn_prop): ... this. --- gdb/ChangeLog | 8 ++++++++ gdb/gdbtypes.c | 15 +++++++-------- gdb/gdbtypes.h | 6 +++--- gdb/value.c | 2 +- 4 files changed, 19 insertions(+), 12 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 857897f8f8..f9ebdd01a7 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,11 @@ +2020-05-07 Simon Marchi + + * gdbtypes.h (struct type) : New method. + (remove_dyn_prop): Remove. Update all users to use + type::remove_dyn_prop. + * gdbtypes.c (remove_dyn_prop): Rename to... + (type::remove_dyn_prop): ... this. + 2020-05-07 Simon Marchi via Gdb-patches * gdbtypes.h (struct type) : New method. diff --git a/gdb/gdbtypes.c b/gdb/gdbtypes.c index 1444351c51..d8b723751e 100644 --- a/gdb/gdbtypes.c +++ b/gdb/gdbtypes.c @@ -2204,7 +2204,7 @@ resolve_dynamic_array_or_string (struct type *type, { if (dwarf2_evaluate_property (prop, NULL, addr_stack, &value)) { - remove_dyn_prop (DYN_PROP_BYTE_STRIDE, type); + type->remove_dyn_prop (DYN_PROP_BYTE_STRIDE); bit_stride = (unsigned int) (value * 8); } else @@ -2621,7 +2621,7 @@ resolve_dynamic_type_internal (struct type *type, if (type_length.has_value ()) { TYPE_LENGTH (resolved_type) = *type_length; - remove_dyn_prop (DYN_PROP_BYTE_SIZE, resolved_type); + resolved_type->remove_dyn_prop (DYN_PROP_BYTE_SIZE); } /* Resolve data_location attribute. */ @@ -2683,27 +2683,26 @@ type::add_dyn_prop (dynamic_prop_node_kind prop_kind, dynamic_prop prop) TYPE_DYN_PROP_LIST (this) = temp; } -/* Remove dynamic property from TYPE in case it exists. */ +/* See gdbtypes.h. */ void -remove_dyn_prop (enum dynamic_prop_node_kind prop_kind, - struct type *type) +type::remove_dyn_prop (dynamic_prop_node_kind kind) { struct dynamic_prop_list *prev_node, *curr_node; - curr_node = TYPE_DYN_PROP_LIST (type); + curr_node = TYPE_DYN_PROP_LIST (this); prev_node = NULL; while (NULL != curr_node) { - if (curr_node->prop_kind == prop_kind) + if (curr_node->prop_kind == kind) { /* Update the linked list but don't free anything. The property was allocated on objstack and it is not known if we are on top of it. Nevertheless, everything is released when the complete objstack is freed. */ if (NULL == prev_node) - TYPE_DYN_PROP_LIST (type) = curr_node->next; + TYPE_DYN_PROP_LIST (this) = curr_node->next; else prev_node->next = curr_node->next; diff --git a/gdb/gdbtypes.h b/gdb/gdbtypes.h index ef991f3c8d..50a0c135de 100644 --- a/gdb/gdbtypes.h +++ b/gdb/gdbtypes.h @@ -884,6 +884,9 @@ struct type This function assumes that this type is objfile-owned. */ void add_dyn_prop (dynamic_prop_node_kind kind, dynamic_prop prop); + /* * Remove dynamic property of kind KIND from this type, if it exists. */ + void remove_dyn_prop (dynamic_prop_node_kind kind); + /* * Type that is a pointer to this type. NULL if no such pointer-to type is known yet. The debugger may add the address of such a type @@ -2103,9 +2106,6 @@ extern struct type *resolve_dynamic_type /* * Predicate if the type has dynamic values, which are not resolved yet. */ extern int is_dynamic_type (struct type *type); -extern void remove_dyn_prop (enum dynamic_prop_node_kind prop_kind, - struct type *type); - extern struct type *check_typedef (struct type *); extern void check_stub_method_group (struct type *, int); diff --git a/gdb/value.c b/gdb/value.c index 7ea39af555..aafbf0fc06 100644 --- a/gdb/value.c +++ b/gdb/value.c @@ -2291,7 +2291,7 @@ set_internalvar (struct internalvar *var, struct value *val) when accessing the value. If we keep it, we would still refer to the origin value. Remove the location property in case it exist. */ - remove_dyn_prop (DYN_PROP_DATA_LOCATION, value_type (new_data.value)); + value_type (new_data.value)->remove_dyn_prop (DYN_PROP_DATA_LOCATION); break; } -- 2.34.1