diff options
author | Mike Frysinger <vapier@gentoo.org> | 2015-03-22 23:10:09 -0400 |
---|---|---|
committer | Mike Frysinger <vapier@gentoo.org> | 2015-11-15 02:30:19 -0500 |
commit | 6e4f085c7f459e0777a71bcb61ed3aa8257fa386 (patch) | |
tree | e2d1bc33b00d55f6765b8cc8cebb41b4cda7cb81 /sim/bfin | |
parent | 1bd1b714219e1c8324cc2fb5feb4496e80b6adfd (diff) | |
download | gdb-6e4f085c7f459e0777a71bcb61ed3aa8257fa386.zip gdb-6e4f085c7f459e0777a71bcb61ed3aa8257fa386.tar.gz gdb-6e4f085c7f459e0777a71bcb61ed3aa8257fa386.tar.bz2 |
sim: sim-close: unify sim_close logic
Other than the nice advantage of all sims having to declare one fewer
common function, this also fixes leakage in pretty much every sim.
Many were not freeing any resources, and a few were inconsistent as
to the ones they did. Now we have a single module that takes care of
all the logic for us.
Most of the non-cgen based ones could be deleted outright. The cgen
ones required adding a callback to the arch-specific cleanup func.
The few that still have close callbacks are to manage their internal
state.
We do not convert erc32, m32c, ppc, rl78, or rx as they do not use
the common sim core.
Diffstat (limited to 'sim/bfin')
-rw-r--r-- | sim/bfin/ChangeLog | 4 | ||||
-rw-r--r-- | sim/bfin/interp.c | 6 |
2 files changed, 4 insertions, 6 deletions
diff --git a/sim/bfin/ChangeLog b/sim/bfin/ChangeLog index 2e62a74..b0205e3 100644 --- a/sim/bfin/ChangeLog +++ b/sim/bfin/ChangeLog @@ -1,3 +1,7 @@ +2015-11-14 Mike Frysinger <vapier@gentoo.org> + + * interp.c (sim_close): Delete. + 2015-10-11 Mike Frysinger <vapier@gentoo.org> PR sim/18407 diff --git a/sim/bfin/interp.c b/sim/bfin/interp.c index 3013ca0..5ba204b 100644 --- a/sim/bfin/interp.c +++ b/sim/bfin/interp.c @@ -820,12 +820,6 @@ sim_open (SIM_OPEN_KIND kind, host_callback *callback, return sd; } -void -sim_close (SIM_DESC sd, int quitting) -{ - sim_module_uninstall (sd); -} - /* Some utils don't like having a NULL environ. */ static const char * const simple_env[] = { "HOME=/", "PATH=/bin", NULL }; |