On 08/11/2011 05:19 AM, Nick Clifton wrote:
> Hi Guys,
> 
>   I am applying the patch below on behalf of Renesas.  It adds support
>   to the RX backend for conditional register moves.
> 
>   Tested without any regressions on an rx-elf toolchain.
> 
> Cheers
>   Nick
> 
> gcc/ChangeLog
> 2011-08-11   Kazuhiro Inaoka  <kazuhiro.inaoka...@renesas.com>
> 
>       * config/rx/rx.md (movsicc): Allow register to register
>       transfers.
>       (*movsicc): Likewise.
>       (*stcc): Restrict this pattern to EQ and NE compares.
>       (*stcc_reg): New pattern.  Works for any comparison but only for
>       register transfers.
> 
> Index: gcc/config/rx/rx.md
> ===================================================================
> --- gcc/config/rx/rx.md       (revision 177584)
> +++ gcc/config/rx/rx.md       (working copy)
> @@ -1598,7 +1598,7 @@
>                  (memex_commutative:SI (match_dup 0)
>                                        (match_dup 2)))
>             (clobber (reg:CC CC_REG))])]
> -  "peep2_regno_dead_p (2, REGNO (operands[0]))"
> +  "peep2_regno_dead_p (2, REGNO (operands[0])) && (optimize < 3 || 
> optimize_size)"

The patch doesn't match the changelog.  And this looks very wrong...

r~

Reply via email to