Re: [PATCH v3] gdb: bfin: new port

[Pedro Alves <pedro at codesourcery dot com>]

On Tuesday 14 December 2010 20:42:26, Mike Frysinger wrote:
> +  /* Pseudo Registers */
> +  BFIN_PC_REGNUM,
> +  BFIN_CC_REGNUM,
> +  BFIN_TEXT_ADDR,              /* Address of .text section.  */
> +  BFIN_TEXT_END_ADDR,          /* Address of the end of .text section.  */
> +  BFIN_DATA_ADDR,              /* Address of .data section.  */
> +
> +  BFIN_FDPIC_EXEC_REGNUM,
> +  BFIN_FDPIC_INTERP_REGNUM,
> +

I thought these (text_addr ... fdpic*) would be removed?

Can you explain why are the PC and CC registers pseudo
registers, but supported as being raw registers anyway?  Couldn't
gdb compute them itself from the other registers, with
gdb's pseudo register support (gdbarch_pseudo_register_read|write)?
googling I found you mentioning that the "CC pseudo register can
be deduced from the ASTAT register", though further googling doesn't
find any mention of what ASTAT is.  I'm sure there's a good reason,
I'm probably just missing a comment somewhere.

-- 
Pedro Alves