On Tue, Feb 28, 2012 at 05:09:05PM -0000, Greta Yorsh wrote:
> Is it OK for GCC 4.7 Stage 4 ?

Technically this is a regression in 4.7 compared to 4.6, 
so I'd like to get this in. 

However given the stage we are and given that it's not a
correctness issue, I would defer to the RMs. 

In any case there are a couple of changes that I'd like done 
as explained below.

>    [(parallel
>      [(set (match_dup 4) (match_op_dup 5 [(match_dup 6) (match_dup 7)]))
>       (clobber (reg:CC CC_REGNUM))])]
> @@ -1180,9 +1181,10 @@
>          (match_operator:SI 5 "commutative_binary_operator"
>           [(match_operand:SI 6 "s_register_operand" "")
>            (match_operand:SI 7 "s_register_operand" "")]))]
> -  "(((operands[6] == operands[0] && operands[7] == operands[1])
> -     || (operands[7] == operands[0] && operands[6] == operands[1]))
> -    && peep2_reg_dead_p (3, operands[0]) && peep2_reg_dead_p (3, 
> operands[1]))"
> +  "(((rtx_equal_p (operands[6], operands[0]) && rtx_equal_p (operands[7], 
> operands[1]))
> +     || (rtx_equal_p (operands[7], operands[0]) && rtx_equal_p (operands[6], 
> operands[1])))

Likewise.

regards
Ramana

Reply via email to