This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: mips-tdep.c: Adjust breakpoints in branch delay slots

From: Daniel Jacobowitz <drow at false dot org>
To: "Maciej W. Rozycki" <macro at mips dot com>
Cc: gdb-patches at sourceware dot org, Chris Dearman <chris at mips dot com>, "Maciej W. Rozycki" <macro at linux-mips dot org>
Date: Fri, 25 May 2007 09:26:46 -0400
Subject: Re: mips-tdep.c: Adjust breakpoints in branch delay slots
References: <Pine.LNX.4.61.0705241853450.21951@perivale.mips.com> <20070524183129.GA10094@caradoc.them.org> <Pine.LNX.4.61.0705251419160.13913@perivale.mips.com>

On Fri, May 25, 2007 at 02:20:55PM +0100, Maciej W. Rozycki wrote:
> > BTW, see mips_single_step_through_delay, which could probably be
> > improved.  Doesn't its comment about MIPS16 having no delay slots
> > disagree with this patch?  Which is right?
> 
>  There are both kinds of jumps/branches in MIPS16e:
> 
> - jal, jalr, jalx and jr do have a delay slot,
> 
> - branches and jalrc and jrc have no delay slot (the two latters are 
>   MIPS16e additions over "traditional" MIPS16).

You might want to fix mips_single_step_through_delay at some point,
then; it doesn't know about any MIPS16 instructions with delay slots.

-- 
Daniel Jacobowitz
CodeSourcery

References:
- mips-tdep.c: Adjust breakpoints in branch delay slots
  - From: Maciej W. Rozycki
- Re: mips-tdep.c: Adjust breakpoints in branch delay slots
  - From: Daniel Jacobowitz
- Re: mips-tdep.c: Adjust breakpoints in branch delay slots
  - From: Maciej W. Rozycki

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]