On 03/17/2011 11:46 AM, Jakub Jelinek wrote:
> - result = gen_rtx_PLUS (tmode, result, tmp);
> + {
> + if (swap_commutative_operands_p (result, tmp))
> + result = gen_rtx_PLUS (tmode, tmp, result);
> + else
> + result = gen_rtx_PLUS (tmode, result, tmp);Hum. What about simplify_gen_binary (PLUS, tmode, result, tmp) which will also do the swap? I'm not wed to this though. I guess I'd be ok with the patch either way. r~
