diff options
author | Tom Tromey <tom@tromey.com> | 2020-10-18 10:47:48 -0600 |
---|---|---|
committer | Tom Tromey <tom@tromey.com> | 2022-07-28 14:16:50 -0600 |
commit | b382c16682acc33d2e81e5604c9dbd408be376d2 (patch) | |
tree | 46ca3ab29dfb4437dd35db2ab605fbc1ebdf1195 /gdb | |
parent | e5213e2c851c295c5e4c3e9b52606c1012029b67 (diff) | |
download | binutils-b382c16682acc33d2e81e5604c9dbd408be376d2.zip binutils-b382c16682acc33d2e81e5604c9dbd408be376d2.tar.gz binutils-b382c16682acc33d2e81e5604c9dbd408be376d2.tar.bz2 |
Change address_space to use new and delete
This changes address_space to use new and delete, and makes some other
small C++-ification changes as well, like changing address_space_num
to be a method.
This patch was needed for the subsequent patch to rewrite the registry
system.
Diffstat (limited to 'gdb')
-rw-r--r-- | gdb/infrun.c | 6 | ||||
-rw-r--r-- | gdb/progspace.c | 38 | ||||
-rw-r--r-- | gdb/progspace.h | 22 | ||||
-rw-r--r-- | gdb/scoped-mock-context.h | 2 | ||||
-rw-r--r-- | gdb/target-debug.h | 2 |
5 files changed, 31 insertions, 39 deletions
diff --git a/gdb/infrun.c b/gdb/infrun.c index 543cccc..033699b 100644 --- a/gdb/infrun.c +++ b/gdb/infrun.c @@ -501,7 +501,7 @@ holding the child stopped. Try \"set detach-on-fork\" or \ } else { - child_inf->aspace = new_address_space (); + child_inf->aspace = new address_space (); child_inf->pspace = new program_space (child_inf->aspace); child_inf->removable = 1; clone_program_space (child_inf->pspace, parent_inf->pspace); @@ -573,7 +573,7 @@ holding the child stopped. Try \"set detach-on-fork\" or \ child_inf->aspace = parent_inf->aspace; child_inf->pspace = parent_inf->pspace; - parent_inf->aspace = new_address_space (); + parent_inf->aspace = new address_space (); parent_inf->pspace = new program_space (parent_inf->aspace); clone_program_space (parent_inf->pspace, child_inf->pspace); @@ -583,7 +583,7 @@ holding the child stopped. Try \"set detach-on-fork\" or \ } else { - child_inf->aspace = new_address_space (); + child_inf->aspace = new address_space (); child_inf->pspace = new program_space (child_inf->aspace); child_inf->removable = 1; child_inf->symfile_flags = SYMFILE_NO_READ; diff --git a/gdb/progspace.c b/gdb/progspace.c index 8735343..c2e2da5 100644 --- a/gdb/progspace.c +++ b/gdb/progspace.c @@ -57,16 +57,10 @@ DEFINE_REGISTRY (address_space, REGISTRY_ACCESS_FIELD) /* Create a new address space object, and add it to the list. */ -struct address_space * -new_address_space (void) +address_space::address_space () + : m_num (++highest_address_space_num) { - struct address_space *aspace; - - aspace = XCNEW (struct address_space); - aspace->num = ++highest_address_space_num; - address_space_alloc_data (aspace); - - return aspace; + address_space_alloc_data (this); } /* Maybe create a new address space object, and add it to the list, or @@ -84,20 +78,12 @@ maybe_new_address_space (void) return program_spaces[0]->aspace; } - return new_address_space (); -} - -static void -free_address_space (struct address_space *aspace) -{ - address_space_free_data (aspace); - xfree (aspace); + return new address_space (); } -int -address_space_num (struct address_space *aspace) +address_space::~address_space () { - return aspace->num; + address_space_free_data (this); } /* Start counting over from scratch. */ @@ -153,7 +139,7 @@ program_space::~program_space () locations for this pspace which we're tearing down. */ clear_symtab_users (SYMFILE_DEFER_BP_RESET); if (!gdbarch_has_shared_address_space (target_gdbarch ())) - free_address_space (this->aspace); + delete this->aspace; /* Discard any data modules have associated with the PSPACE. */ program_space_free_data (this); } @@ -411,17 +397,17 @@ update_address_spaces (void) if (shared_aspace) { - struct address_space *aspace = new_address_space (); + struct address_space *aspace = new address_space (); - free_address_space (current_program_space->aspace); + delete current_program_space->aspace; for (struct program_space *pspace : program_spaces) pspace->aspace = aspace; } else for (struct program_space *pspace : program_spaces) { - free_address_space (pspace->aspace); - pspace->aspace = new_address_space (); + delete pspace->aspace; + pspace->aspace = new address_space (); } for (inferior *inf : all_inferiors ()) @@ -459,5 +445,5 @@ initialize_progspace (void) modules have done that. Do this before initialize_current_architecture, because that accesses the ebfd of current_program_space. */ - current_program_space = new program_space (new_address_space ()); + current_program_space = new program_space (new address_space ()); } diff --git a/gdb/progspace.h b/gdb/progspace.h index 662e569..8531da4 100644 --- a/gdb/progspace.h +++ b/gdb/progspace.h @@ -385,10 +385,22 @@ private: associating caches to each address space. */ struct address_space { - int num; + /* Create a new address space object, and add it to the list. */ + address_space (); + ~address_space (); + DISABLE_COPY_AND_ASSIGN (address_space); + + /* Returns the integer address space id of this address space. */ + int num () const + { + return m_num; + } /* Per aspace data-pointers required by other GDB modules. */ - REGISTRY_FIELDS; + REGISTRY_FIELDS {}; + +private: + int m_num; }; /* The list of all program spaces. There's always at least one. */ @@ -430,17 +442,11 @@ private: program_space *m_saved_pspace; }; -/* Create a new address space object, and add it to the list. */ -extern struct address_space *new_address_space (void); - /* Maybe create a new address space object, and add it to the list, or return a pointer to an existing address space, in case inferiors share an address space. */ extern struct address_space *maybe_new_address_space (void); -/* Returns the integer address space id of ASPACE. */ -extern int address_space_num (struct address_space *aspace); - /* Update all program spaces matching to address spaces. The user may have created several program spaces, and loaded executables into them before connecting to the target interface that will create the diff --git a/gdb/scoped-mock-context.h b/gdb/scoped-mock-context.h index b91d43b..a989530 100644 --- a/gdb/scoped-mock-context.h +++ b/gdb/scoped-mock-context.h @@ -38,7 +38,7 @@ struct scoped_mock_context Target mock_target; ptid_t mock_ptid {1, 1}; - program_space mock_pspace {new_address_space ()}; + program_space mock_pspace {new address_space ()}; inferior mock_inferior {mock_ptid.pid ()}; thread_info mock_thread {&mock_inferior, mock_ptid}; diff --git a/gdb/target-debug.h b/gdb/target-debug.h index c2b1db1..ab89c0a 100644 --- a/gdb/target-debug.h +++ b/gdb/target-debug.h @@ -89,7 +89,7 @@ #define target_debug_print_LONGEST_p(X) \ target_debug_do_print (phex (*(X), 0)) #define target_debug_print_struct_address_space_p(X) \ - target_debug_do_print (plongest (address_space_num (X))) + target_debug_do_print (plongest ((X)->num ())) #define target_debug_print_struct_bp_target_info_p(X) \ target_debug_do_print (core_addr_to_string ((X)->placed_address)) #define target_debug_print_struct_expression_p(X) \ |