Re: [RFA v2] arm-pikeos: software single step

[Simon Marchi <simon dot marchi at polymtl dot ca>]

(not sure why the From address is y@eu.adacore.com, I'll replace it with 
Joel's).

> +  symbol_table = (asymbol **) xmalloc (storage_needed);
> +  make_cleanup (xfree, symbol_table);

It would make Tom really happy if you could avoid introducing a cleanup 
:).  I would suggest using an std::vector<asymbol *>.  If you'd rather 
not do it it's not a big deal, we can change it after.

> diff --git a/gdb/configure.tgt b/gdb/configure.tgt
> index 6d1a4df..a1f703f 100644
> --- a/gdb/configure.tgt
> +++ b/gdb/configure.tgt
> @@ -180,6 +180,7 @@ arm*-*-symbianelf*)
>  	;;
>  arm*-*-*)
>  	# Target: ARM embedded system
> +        gdb_target_obs="arm-pikeos-tdep.o"
>  	gdb_sim=../sim/arm/libsim.a
>  	;;

The object should be added to ALL_TARGETS_OBS in the Makefile, so it's 
included in an enable-targets=all build.

Just to understand better why the object file needs to be added here, 
what is the target triplet? arm-pikeos-something, arm-none-something?

Just wondering: when you connect to the target, you talk to a gdb stub 
that is built right into the application/rtos?

Simon