Re: [patch, mips] Micromips delay slot fix

2013-06-12 Thread Richard Sandiford
Richard Sandiford rdsandif...@googlemail.com writes: Moore, Catherine catherine_mo...@mentor.com writes: I'm testing a slightly different patch from the one that Steve posted: Index: mips.md === --- mips.md (revision 199648)

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 Tue, 11 Jun 2013, Steve Ellcey

Re: [patch, mips] Micromips delay slot fix

2013-06-11 Thread Richard Sandiford
Moore, Catherine catherine_mo...@mentor.com 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 @@

[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

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 to

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 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 microMIPS patches