aboutsummaryrefslogtreecommitdiff
path: root/sim/lm32
diff options
context:
space:
mode:
authorMike Frysinger <vapier@gentoo.org>2015-12-26 07:05:41 -0500
committerMike Frysinger <vapier@gentoo.org>2015-12-26 07:19:07 -0500
commit0e9672991e9346d5c2cb9cd33e30fb3e573f88b8 (patch)
treeb9a00b51355ac7ebed43ebf97f78b7a373b42936 /sim/lm32
parentf66affe97c40465e70249970b3339cc2f0a40e69 (diff)
downloadgdb-0e9672991e9346d5c2cb9cd33e30fb3e573f88b8.zip
gdb-0e9672991e9346d5c2cb9cd33e30fb3e573f88b8.tar.gz
gdb-0e9672991e9346d5c2cb9cd33e30fb3e573f88b8.tar.bz2
sim: standardize sim_create_inferior handling of argv a bit more
For targets that process argv in sim_create_inferior, improve the code: - provide more details in the comment - make the check for when to re-init more robust - clean out legacy sim_copy_argv code This will be cleaned up more in the future when we have a common inferior creation function, but at least help new ports get it right until then.
Diffstat (limited to 'sim/lm32')
-rw-r--r--sim/lm32/ChangeLog4
-rw-r--r--sim/lm32/sim-if.c13
2 files changed, 13 insertions, 4 deletions
diff --git a/sim/lm32/ChangeLog b/sim/lm32/ChangeLog
index b94ef40..a3da560 100644
--- a/sim/lm32/ChangeLog
+++ b/sim/lm32/ChangeLog
@@ -1,3 +1,7 @@
+2015-12-26 Mike Frysinger <vapier@gentoo.org>
+
+ * sim-if.c (sim_create_inferior): Replace old #if 0 code with dupargv.
+
2015-12-25 Mike Frysinger <vapier@gentoo.org>
* Makefile.in (SIM_OBJS): Delete sim-model.o.
diff --git a/sim/lm32/sim-if.c b/sim/lm32/sim-if.c
index 3b0575a..eb13363 100644
--- a/sim/lm32/sim-if.c
+++ b/sim/lm32/sim-if.c
@@ -263,10 +263,15 @@ sim_create_inferior (sd, abfd, argv, envp)
addr = 0;
sim_pc_set (current_cpu, addr);
-#if 0
- STATE_ARGV (sd) = sim_copy_argv (argv);
- STATE_ENVP (sd) = sim_copy_argv (envp);
-#endif
+ /* Standalone mode (i.e. `run`) will take care of the argv for us in
+ sim_open() -> sim_parse_args(). But in debug mode (i.e. 'target sim'
+ with `gdb`), we need to handle it because the user can change the
+ argv on the fly via gdb's 'run'. */
+ if (STATE_PROG_ARGV (sd) != argv)
+ {
+ freeargv (STATE_PROG_ARGV (sd));
+ STATE_PROG_ARGV (sd) = dupargv (argv);
+ }
return SIM_RC_OK;
}