From a664f67e50eff30198097d51cec0ec4690abb2a1 Mon Sep 17 00:00:00 2001 From: Simon Marchi Date: Sat, 21 Oct 2017 12:06:42 -0400 Subject: Get rid of VEC (mem_region) This patch removes VEC (mem_region). Doing so requires touching a lot of little things here and there. The fields in mem_attrib are now initialized during construction. The values match those that were in default_mem_attrib (now removed). unknown_mem_attrib is also removed, and replaced with a static method (mem_attrib::unknown) that returns the equivalent. mem_region is initialized in a way similar to mem_region_init (now removed) did. I found the organization of mem_region_list and target_mem_region_list a bit confusing. Sometimes mem_region_list points to the same vector as target_mem_region_list (and therefore does not own it), and sometimes (when the user manually edits the mem regions) points to another vector, and in this case owns it. To avoid this ambiguity, I think it is simpler to have two vectors, one for target-defined regions and one for user-defined regions, and have mem_region_list point to one or the other. There are now no vector objects dynamically allocated, both are static. The make-target-delegates script does not generate valid code when a target method returns a type with a parameter list. For this reason, I created a typedef (mem_region_vector) that's only used in the target_ops structure. If you speak perl, you are welcome to improve the script! Regtested on the buildbot. gdb/ChangeLog: * memattr.h: Don't include vec.h. (struct mem_attrib): Initialize fields. : New static method. (struct mem_region): Add constructors, operator<, initialize fields. * memattr.c: Include algorithm. (default_mem_attrib, unknown_mem_attrib): Remove. (user_mem_region_list): New global. (target_mem_region_list, mem_region_list): Change type to std::vector. (mem_use_target): Now a function. (target_mem_regions_valid): Change type to bool. (mem_region_lessthan, mem_region_cmp, mem_region_init): Remove. (require_user_regions): Adjust. (require_target_regions): Adjust. (create_mem_region): Adjust. (lookup_mem_region): Adjust. (invalidate_target_mem_regions): Adjust. (mem_clear): Rename to... (user_mem_clear): ... this, and adjust. (mem_command): Adjust. (info_mem_command): Adjust. (mem_enable, enable_mem_command, mem_disable, disable_mem_command): Adjust. (mem_delete): Adjust. (delete_mem_command): Adjust. * memory-map.h (parse_memory_map): Return an std::vector. * memory-map.c (parse_memory_map): Likewise. (struct memory_map_parsing_data): Add constructor. : Point to std::vector. (memory_map_start_memory): Adjust. (memory_map_end_memory): Adjust. (memory_map_end_property): Adjust. (clear_result): Remove. * remote.c (remote_memory_map): Return an std::vector. * target-debug.h (target_debug_print_VEC_mem_region_s__p): Remove. (target_debug_print_mem_region_vector): New. * target-delegates.c: Regenerate. * target.h (mem_region_vector): New typedef. (to_memory_map): Return mem_region_vector. (target_memory_map): Return an std::vector. * target.c (target_memory_map): Return an std::vector. (flash_erase_command): Adjust. --- gdb/target.c | 44 ++++++++++++++++++-------------------------- 1 file changed, 18 insertions(+), 26 deletions(-) (limited to 'gdb/target.c') diff --git a/gdb/target.c b/gdb/target.c index 6fa9d59..bd81fcd 100644 --- a/gdb/target.c +++ b/gdb/target.c @@ -1479,34 +1479,31 @@ target_write_raw_memory (CORE_ADDR memaddr, const gdb_byte *myaddr, ssize_t len) /* Fetch the target's memory map. */ -VEC(mem_region_s) * +std::vector target_memory_map (void) { - VEC(mem_region_s) *result; - struct mem_region *last_one, *this_one; - int ix; - result = current_target.to_memory_map (¤t_target); - if (result == NULL) - return NULL; + std::vector result + = current_target.to_memory_map (¤t_target); + if (result.empty ()) + return result; - qsort (VEC_address (mem_region_s, result), - VEC_length (mem_region_s, result), - sizeof (struct mem_region), mem_region_cmp); + std::sort (result.begin (), result.end ()); /* Check that regions do not overlap. Simultaneously assign a numbering for the "mem" commands to use to refer to each region. */ - last_one = NULL; - for (ix = 0; VEC_iterate (mem_region_s, result, ix, this_one); ix++) + mem_region *last_one = NULL; + for (size_t ix = 0; ix < result.size (); ix++) { + mem_region *this_one = &result[ix]; this_one->number = ix; - if (last_one && last_one->hi > this_one->lo) + if (last_one != NULL && last_one->hi > this_one->lo) { warning (_("Overlapping regions in memory map: ignoring")); - VEC_free (mem_region_s, result); - return NULL; + return std::vector (); } + last_one = this_one; } @@ -3815,30 +3812,25 @@ flash_erase_command (char *cmd, int from_tty) { /* Used to communicate termination of flash operations to the target. */ bool found_flash_region = false; - struct mem_region *m; struct gdbarch *gdbarch = target_gdbarch (); - VEC(mem_region_s) *mem_regions = target_memory_map (); + std::vector mem_regions = target_memory_map (); /* Iterate over all memory regions. */ - for (int i = 0; VEC_iterate (mem_region_s, mem_regions, i, m); i++) + for (const mem_region &m : mem_regions) { - /* Fetch the memory attribute. */ - struct mem_attrib *attrib = &m->attrib; - /* Is this a flash memory region? */ - if (attrib->mode == MEM_FLASH) + if (m.attrib.mode == MEM_FLASH) { found_flash_region = true; - target_flash_erase (m->lo, m->hi - m->lo); + target_flash_erase (m.lo, m.hi - m.lo); ui_out_emit_tuple tuple_emitter (current_uiout, "erased-regions"); current_uiout->message (_("Erasing flash memory region at address ")); - current_uiout->field_fmt ("address", "%s", paddress (gdbarch, - m->lo)); + current_uiout->field_fmt ("address", "%s", paddress (gdbarch, m.lo)); current_uiout->message (", size = "); - current_uiout->field_fmt ("size", "%s", hex_string (m->hi - m->lo)); + current_uiout->field_fmt ("size", "%s", hex_string (m.hi - m.lo)); current_uiout->message ("\n"); } } -- cgit v1.1