From e947a8482ac8ea529869c15b6949e4f19272c747 Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Wed, 23 Sep 2020 09:32:54 -0600 Subject: [PATCH] Introduce and use compile_module_up This introduces compile_module_up, a unique pointer for compile_module, and changes a few spots to use it. gdb/ChangeLog 2020-09-23 Tom Tromey * compile/compile.c (eval_compile_command): Update. * compile/compile-object-run.h (compile_object_run): Take a compile_module_up. * compile/compile-object-run.c (compile_object_run): Take a compile_module_up. * compile/compile-object-load.h (struct compile_module): Add constructor, destructor. (compile_module_up): New typedef. (compile_object_load): Return compile_object_up. * compile/compile-object-load.c (compile_object_load): Return compile_module_up. --- gdb/ChangeLog | 14 ++++++++++++++ gdb/compile/compile-object-load.c | 5 ++--- gdb/compile/compile-object-load.h | 12 +++++++++++- gdb/compile/compile-object-run.c | 4 +--- gdb/compile/compile-object-run.h | 2 +- gdb/compile/compile.c | 7 +++---- 6 files changed, 32 insertions(+), 12 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index f3799059aa4..6f89a05e451 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,17 @@ +2020-09-23 Tom Tromey + + * compile/compile.c (eval_compile_command): Update. + * compile/compile-object-run.h (compile_object_run): Take a + compile_module_up. + * compile/compile-object-run.c (compile_object_run): Take a + compile_module_up. + * compile/compile-object-load.h (struct compile_module): Add + constructor, destructor. + (compile_module_up): New typedef. + (compile_object_load): Return compile_object_up. + * compile/compile-object-load.c (compile_object_load): Return + compile_module_up. + 2020-09-23 Tom Tromey * compile/compile-object-run.c (struct do_module_cleanup): Add diff --git a/gdb/compile/compile-object-load.c b/gdb/compile/compile-object-load.c index 2f416079025..dff10fe94f4 100644 --- a/gdb/compile/compile-object-load.c +++ b/gdb/compile/compile-object-load.c @@ -582,7 +582,7 @@ store_regs (struct type *regs_type, CORE_ADDR regs_base) COMPILE_I_PRINT_ADDRESS_SCOPE when COMPILE_I_PRINT_VALUE_SCOPE should have been used instead. */ -struct compile_module * +compile_module_up compile_object_load (const compile_file_names &file_names, enum compile_i_scope_types scope, void *scope_data) { @@ -594,7 +594,6 @@ compile_object_load (const compile_file_names &file_names, long storage_needed; asymbol **symbol_table, **symp; long number_of_symbols, missing_symbols; - struct compile_module *retval; struct type *regs_type, *out_value_type = NULL; char **matching; struct objfile *objfile; @@ -790,7 +789,7 @@ compile_object_load (const compile_file_names &file_names, paddress (target_gdbarch (), out_value_addr)); } - retval = XNEW (struct compile_module); + compile_module_up retval (new struct compile_module); retval->objfile = objfile_holder.release (); retval->source_file = xstrdup (file_names.source_file ()); retval->func_sym = func_sym; diff --git a/gdb/compile/compile-object-load.h b/gdb/compile/compile-object-load.h index c4adc719141..9def29ebc9b 100644 --- a/gdb/compile/compile-object-load.h +++ b/gdb/compile/compile-object-load.h @@ -46,6 +46,13 @@ private: struct compile_module { + compile_module () = default; + + DISABLE_COPY_AND_ASSIGN (compile_module); + + compile_module &operator= (compile_module &&other) = default; + compile_module (compile_module &&other) = default; + /* objfile for the compiled module. */ struct objfile *objfile; @@ -77,7 +84,10 @@ struct compile_module struct munmap_list *munmap_list_head; }; -extern struct compile_module *compile_object_load +/* A unique pointer for a compile_module. */ +typedef std::unique_ptr compile_module_up; + +extern compile_module_up compile_object_load (const compile_file_names &fnames, enum compile_i_scope_types scope, void *scope_data); diff --git a/gdb/compile/compile-object-run.c b/gdb/compile/compile-object-run.c index 31731b466c9..4c98be9cc1a 100644 --- a/gdb/compile/compile-object-run.c +++ b/gdb/compile/compile-object-run.c @@ -131,7 +131,7 @@ create_copied_type_recursive (objfile *objfile, type *func_type) longer touch MODULE's memory after this function has been called. */ void -compile_object_run (struct compile_module *module) +compile_object_run (compile_module_up &&module) { struct value *func_val; struct do_module_cleanup *data; @@ -152,8 +152,6 @@ compile_object_run (struct compile_module *module) data->munmap_list_head = module->munmap_list_head; xfree (module->source_file); - xfree (module); - module = NULL; try { diff --git a/gdb/compile/compile-object-run.h b/gdb/compile/compile-object-run.h index 9311d02c379..45375db654f 100644 --- a/gdb/compile/compile-object-run.h +++ b/gdb/compile/compile-object-run.h @@ -19,6 +19,6 @@ #include "compile-object-load.h" -extern void compile_object_run (struct compile_module *module); +extern void compile_object_run (compile_module_up &&module); #endif /* COMPILE_COMPILE_OBJECT_RUN_H */ diff --git a/gdb/compile/compile.c b/gdb/compile/compile.c index 8d8c2b14eba..6e8ec40942e 100644 --- a/gdb/compile/compile.c +++ b/gdb/compile/compile.c @@ -820,14 +820,13 @@ void eval_compile_command (struct command_line *cmd, const char *cmd_string, enum compile_i_scope_types scope, void *scope_data) { - struct compile_module *compile_module; - compile_file_names fnames = compile_to_object (cmd, cmd_string, scope); gdb::unlinker object_remover (fnames.object_file ()); gdb::unlinker source_remover (fnames.source_file ()); - compile_module = compile_object_load (fnames, scope, scope_data); + compile_module_up compile_module = compile_object_load (fnames, scope, + scope_data); if (compile_module == NULL) { gdb_assert (scope == COMPILE_I_PRINT_ADDRESS_SCOPE); @@ -840,7 +839,7 @@ eval_compile_command (struct command_line *cmd, const char *cmd_string, source_remover.keep (); object_remover.keep (); - compile_object_run (compile_module); + compile_object_run (std::move (compile_module)); } /* See compile/compile-internal.h. */ -- 2.30.2