Re: [RFA 6/8] New port: TI C6x: gdbserver

[Pedro Alves <pedro@codesourcery.com>]

On Thursday 21 July 2011 08:31:19, Yao Qi wrote:
> >> +<!DOCTYPE feature SYSTEM "gdb-target.dtd">
> >> +<feature name="org.gnu.gdb.tic6x.cpu">
> >> +  <reg name="A0" bitsize="32"/>
> >> +  <reg name="A1" bitsize="32"/>
> >> +  <reg name="A2" bitsize="32"/>
> > 
> > ...
> > 
> > These are all general purpose, core registers, right?
> > 
> 
> Not really.  A0-31 and B0-31 are general purpose registers, even there
> are some difference among different C6x cores (such as C62x, C67x, and
> C64x).  The rest of them can be regarded as control registers and status
> registers.  The target description should be refined for the different
> C6x cores, and I'd like to do it in follow-up patches later.  Is it OK?

I'd rather start with things clean than change the target's register
set after the fact, but I won't insist.  At least you should document in
the manual which registers the new standard feature org.gnu.gdb.tic6x.cpu
requires (under Standard Target Features) though.

-- 
Pedro Alves