From 283a99589a0780a3b1fc011885b630bd9223ebba Mon Sep 17 00:00:00 2001 From: Joel Brobecker Date: Tue, 24 Mar 2015 11:14:13 -0700 Subject: Do not make "prop" field of struct dynamic_prop_list a pointer. struct dynamic_prop_list is declared as follow: struct dynamic_prop_list { [...] /* The dynamic property itself. */ struct dynamic_prop *prop; [...] }; In this case, the pointer indirection is unnecessary and costing us, for each dynamic property, the memory needed to store one pointer. This patch removes this pointer indirection, savin us a tiny bit of memory, as well as reduces a bit the complexity by removing the need to allocate memory for the property, as the allocation is now part of the struct itself. gdb/ChangeLog: * gdbtypes.h (struct dynamic_prop_list) : Remove pointer indirection. * gdbtypes.c (get_dyn_prop): Adjust, following change above. (add_dyn_prop, copy_dynamic_prop_list): Likewise. Tested on x86_64-linux. --- gdb/ChangeLog | 7 +++++++ 1 file changed, 7 insertions(+) (limited to 'gdb/ChangeLog') diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 78ccc8a..43dda55 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,12 @@ 2015-03-24 Joel Brobecker + * gdbtypes.h (struct dynamic_prop_list) : Remove + pointer indirection. + * gdbtypes.c (get_dyn_prop): Adjust, following change above. + (add_dyn_prop, copy_dynamic_prop_list): Likewise. + +2015-03-24 Joel Brobecker + * gdbtypes.h (enum dynamic_prop_node_kind) : Renames DYN_ATTR_DATA_LOCATION. (TYPE_DATA_LOCATION): Use DYN_PROP_DATA_LOCATION instead of -- cgit v1.1