Re: [PATCH][ARM] minmax_arithsi for non-canonical operand order with MINUS operator

2013-04-09 Thread Richard Earnshaw
On 02/04/13 17:06, Kyrylo Tkachov wrote: From: Ramana Radhakrishnan [mailto:ramana@googlemail.com] Sent: 02 April 2013 11:10 To: Kyrylo Tkachov Cc: gcc-patches@gcc.gnu.org; Richard Earnshaw; Ramana Radhakrishnan Subject: Re: [PATCH][ARM] minmax_arithsi for non-canonical operand order

Re: [PATCH][ARM] minmax_arithsi for non-canonical operand order with MINUS operator

2013-04-02 Thread Ramana Radhakrishnan
On Thu, Mar 21, 2013 at 6:09 PM, Kyrylo Tkachov kyrylo.tkac...@arm.com wrote: Hi all, This patch adds a splitter variant of the minmax_arithsi pattern for when the operator is non-commutative (MINUS) and the ordering of the operands is not canonical. That is, it will trigger for: #define

RE: [PATCH][ARM] minmax_arithsi for non-canonical operand order with MINUS operator

2013-04-02 Thread Kyrylo Tkachov
From: Ramana Radhakrishnan [mailto:ramana@googlemail.com] Sent: 02 April 2013 11:10 To: Kyrylo Tkachov Cc: gcc-patches@gcc.gnu.org; Richard Earnshaw; Ramana Radhakrishnan Subject: Re: [PATCH][ARM] minmax_arithsi for non-canonical operand order with MINUS operator On Thu, Mar 21, 2013

[PATCH][ARM] minmax_arithsi for non-canonical operand order with MINUS operator

2013-03-21 Thread Kyrylo Tkachov
Hi all, This patch adds a splitter variant of the minmax_arithsi pattern for when the operator is non-commutative (MINUS) and the ordering of the operands is not canonical. That is, it will trigger for: #define MAX(a, b) (a b ? a : b) int foo (int a, int b, int c) { return c - MAX (a,b); }