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 with

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 operato

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 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 MAX(a, b) (a >

[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); } a