From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 1177 invoked by alias); 8 Jul 2008 14:59:36 -0000 Received: (qmail 1165 invoked by uid 22791); 8 Jul 2008 14:59:35 -0000 X-Spam-Check-By: sourceware.org Received: from snape.ecoscentric.com (HELO snape.ecoscentric.com) (212.13.207.199) by sourceware.org (qpsmtpd/0.31) with ESMTP; Tue, 08 Jul 2008 14:58:57 +0000 Received: from localhost (snape.ecoscentric.com [127.0.0.1]) by snape.ecoscentric.com (Postfix) with ESMTP id 0E3EDDC8DD1; Tue, 8 Jul 2008 15:58:54 +0100 (BST) Received: from snape.ecoscentric.com ([127.0.0.1]) by localhost (snape.ecoscentric.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id The6CUGi9f1g; Tue, 8 Jul 2008 15:58:49 +0100 (BST) Message-ID: <487380A9.2080300@eCosCentric.com> Date: Tue, 08 Jul 2008 14:59:00 -0000 From: Jonathan Larmour User-Agent: Thunderbird 1.5.0.12 (X11/20070530) MIME-Version: 1.0 To: luisgpm@linux.vnet.ibm.com CC: gdb-patches@sourceware.org Subject: Re: [PATCH] PPC - Stepping off breakpoints in non-stop mode References: <4872C52C.1020701@eCosCentric.com> <1215489163.5333.29.camel@gargoyle> In-Reply-To: <1215489163.5333.29.camel@gargoyle> OpenPGP: id=A5FB74E6 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2008-07/txt/msg00115.txt.bz2 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