Hi,

Ulrich Weigand schrieb:
> Daniel Jacobowitz wrote:
>> On Wed, Aug 08, 2007 at 08:44:29AM +0200, Markus Deuling wrote:
>>> +  if (target_has_registers && target_has_stack && target_has_memory
>>> +      /* FIXME: Currently needed for dwarf2_read_address to work.  */
>>> +      && type != builtin_type_uint32)
>> I think that you shouldn't commit a patch with this hack in it.
>> It looks very fragile.
> 
> Agreed.  Markus, this is a special hack for the combined PPE/SPE
> debugger, it shouldn't be necessary for the SPU-standalone (and
> mainline) version.
> 
> I'll have to look into fixing the underlying issue cleanly when
> merging the combined debugger ...

ok, thank you. I attached a patch with a SPU-only version. This can
later be replaced by the version suitable for the combined debugger.

Tested on spu-elf without regression.

ChangeLog:

	* spu-tdep.c (spu_pointer_to_address): New function.
	(spu_integer_to_address): Likewise.
	(spu_gdbarch_init): Add spu_pointer_to_address and 
	spu_integer_to_address to gdbarch.

Is this ok to commit?

-- 
Markus Deuling
GNU Toolchain for Linux on Cell BE
deuling@de.ibm.com