Re: [patch, mips] Micromips delay slot fix

2013-06-12 Thread Richard Sandiford
Richard Sandiford writes: > "Moore, Catherine" writes: >> I'm testing a slightly different patch from the one that Steve posted: >> >> Index: mips.md >> === >> --- mips.md (revision 199648) >> +++ mips.md (working copy) >> @@

Re: [patch, mips] Micromips delay slot fix

2013-06-11 Thread Richard Sandiford
"Moore, Catherine" writes: > I'm testing a slightly different patch from the one that Steve posted: > > Index: mips.md > === > --- mips.md (revision 199648) > +++ mips.md (working copy) > @@ -703,8 +703,13 @@ > > ;; Is it a s

RE: [patch, mips] Micromips delay slot fix

2013-06-11 Thread Moore, Catherine
> -Original Message- > From: Maciej W. Rozycki [mailto:ma...@codesourcery.com] > Sent: Monday, June 10, 2013 8:50 PM > To: Steve Ellcey; Moore, Catherine > Cc: Richard Sandiford; gcc-patches@gcc.gnu.org > Subject: Re: [patch, mips] Micromips delay slot fix > > On

Re: [patch, mips] Micromips delay slot fix

2013-06-10 Thread Maciej W. Rozycki
On Tue, 11 Jun 2013, Steve Ellcey wrote: > Sorry, it should have been 'main(void) {return 0; }. Then you get (with > the patch): > > j $31 > move$2,$0 > > instead of: > > move$2,$0 > j $31 Hmm, something must have been missed then from the microMIP

Re: [patch, mips] Micromips delay slot fix

2013-06-10 Thread Steve Ellcey
On Mon, 2013-06-10 at 23:13 +0100, Maciej W. Rozycki wrote: > On Mon, 10 Jun 2013, Steve Ellcey wrote: > > > We found a bug in the micromips implementation where the branch delay slot > > was not getting filled for micromips. You can reproduce this with a program > > that just has an empty main

Re: [patch, mips] Micromips delay slot fix

2013-06-10 Thread Maciej W. Rozycki
On Mon, 10 Jun 2013, Steve Ellcey wrote: > We found a bug in the micromips implementation where the branch delay slot > was not getting filled for micromips. You can reproduce this with a program > that just has an empty main function. Andrew Bennett created this fix for > it and we would like

[patch, mips] Micromips delay slot fix

2013-06-10 Thread Steve Ellcey
We found a bug in the micromips implementation where the branch delay slot was not getting filled for micromips. You can reproduce this with a program that just has an empty main function. Andrew Bennett created this fix for it and we would like to check it in. I am submitting it for Andrew sinc