This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Commit: ARM SIM: Add tracing support.


On Fri, Mar 14, 2014 at 10:23 AM, Joel Brobecker <brobecker@adacore.com> wrote:
> Hey Nick,
>
>>   I am applying the attached patch to add tracing support to the ARM
>>   simulator.  With the patch applied three new command line options are
>>   now supported: -t enables tracing, -d enables disassembly and -z
>>   enables a limited form of tracing which just reports function entry
>>   and exits.
>>
>>   In the course of testing this patch I also came across a variety of
>>   compile time warning messages.  So the patch also tidies these up,
>>   just to make things clean and proper.
>
> I haven't looked at the patch itself (I trust you on this part).
> The part that I wondered about is documentation. I don't think
> we have a manual for the embedded simulator in general, but IWBN
> if we could somehow document those switches. Perhaps the best we could
> do is just at least announce them in the gdb/NEWS file?

fwiw, I think putting such effort into gdb's sims is more effort than
it's worth.

But if the community wants it, can we use something like sim/NEWS instead?

> I assume that "target sim --help" will print a usage, so people have
> a way of get a list of supported switches without having to read
> the code?

Yeah, the patch has that.

Nick, can we tighten up the argument parsing here?
IIUC, -tfoo will get recognized as -t.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]