Re: MIPS Fix PR18141

2011-09-27 Thread Richard Sandiford
Dinar Temirbulatov writes: > I prepared the new version of patch, re-tested new version on again on > mipsel-unknown-linux-gnu with no new regressions. Thanks, looks good. I'll apply once the copyright assignment comes through. Richard

Re: MIPS Fix PR18141

2011-09-26 Thread Dinar Temirbulatov
hi, Richard On Mon, Sep 26, 2011 at 11:50 AM, Richard Sandiford wrote: > Dinar Temirbulatov writes: >> I found typo in the patch instead of checking *set_after != 0 it was >> set_after != 0,  here is corrected version of patch. I retested the >> patch without typo on mipsel-unknown-linux-gnu with

Re: MIPS Fix PR18141

2011-09-26 Thread Richard Sandiford
Dinar Temirbulatov writes: > I found typo in the patch instead of checking *set_after != 0 it was > set_after != 0, here is corrected version of patch. I retested the > patch without typo on mipsel-unknown-linux-gnu with no new > regressions. But my point was that *set_after should always be non

Re: MIPS Fix PR18141

2011-09-23 Thread Dinar Temirbulatov
Hi, I found typo in the patch instead of checking *set_after != 0 it was set_after != 0, here is corrected version of patch. I retested the patch without typo on mipsel-unknown-linux-gnu with no new regressions. thanks, Dinar. On Fri, Sep 23, 2011 at 3:11 PM

Re: MIPS Fix PR18141

2011-09-23 Thread Dinar Temirbulatov
hi, Richard, This version of patch showed no regressions on mipsel-unknown-linux-gnu. Thanks, Dinar. On Wed, Sep 21, 2011 at 4:59 PM, Richard Sandiford wrote: > Dinar Temirbulatov writes: >> @@ -14696,7 +14696,11 @@ mips_avoid_hazard (rtx after, rtx insn, >>       clobb

Re: MIPS Fix PR18141

2011-09-21 Thread Richard Sandiford
Dinar Temirbulatov writes: > @@ -14696,7 +14696,11 @@ mips_avoid_hazard (rtx after, rtx insn, > clobber hi and lo. */ >if (*hilo_delay < 2 && reg_set_p (lo_reg, pattern)) > nops = 2 - *hilo_delay; > - else if (*delayed_reg != 0 && reg_referenced_p (*delayed_reg, pattern)) > + els