This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] PPC - Stepping off breakpoints in non-stop mode
- From: Jonathan Larmour <jifl at eCosCentric dot com>
- To: luisgpm at linux dot vnet dot ibm dot com
- Cc: gdb-patches at sourceware dot org
- Date: Tue, 08 Jul 2008 15:58:49 +0100
- Subject: Re: [PATCH] PPC - Stepping off breakpoints in non-stop mode
- Openpgp: id=A5FB74E6
- References: <4872C52C.1020701@eCosCentric.com> <1215489163.5333.29.camel@gargoyle>
Luis Machado wrote:
> Hi Jonathan,
>
> Is this an embedded target?
Yes, powerpc-eabi
Jifl
>> Unfortunately this fails to build due to this warning (in combination with
>> -Werror):
>> .../rs6000-tdep.c: In function âppc_displaced_step_fixupâ:
>> .../rs6000-tdep.c:885: warning: passing argument 3 of
>> âregcache_cooked_read_unsignedâ from incompatible pointer type
>> make[2]: *** [rs6000-tdep.o] Error 1
>>
>> This is because in the following line, current_pc is CORE_ADDR, but
>> regcache_cooked_read_unsigned's last arg is meant to be a ULONGEST *.
>>
>> + regcache_cooked_read_unsigned (regs, gdbarch_pc_regnum (gdbarch),
>> + ¤t_pc);
>>
>> I was thinking of just casting it, but in fact would it be better to be
>> calling regcache_read_pc() here as well as regcache_write_pc() in the rest
>> of the patch when operating on the pc? From earlier in this thread, there
>> was a little discussion about address sizes, so I thought I'd best ask
>> before submitting a patch, as there could be portability implications and
>> it might have been done this way for a reason.
>>
>> Jifl
>
>
--
eCosCentric Limited http://www.eCosCentric.com/ The eCos experts
Barnwell House, Barnwell Drive, Cambridge, UK. Tel: +44 1223 245571
Registered in England and Wales: Reg No 4422071.
------["Si fractum non sit, noli id reficere"]------ Opinions==mine