Re: [PATCH] ARM: Don't clobber CC reg when it is live after the peephole window

2013-06-19 Thread Richard Earnshaw
On 18/06/13 17:22, Meador Inge wrote: Ping. On 06/06/2013 01:23 PM, Meador Inge wrote: On 06/06/2013 08:11 AM, Richard Earnshaw wrote: I understand (and agree with) this bit... +(define_peephole2 + [(set (reg:CC CC_REGNUM) +(compare:CC (match_operand:SI 0 "register_operand" "") +

Re: [PATCH] ARM: Don't clobber CC reg when it is live after the peephole window

2013-06-18 Thread Meador Inge
Ping. On 06/06/2013 01:23 PM, Meador Inge wrote: > On 06/06/2013 08:11 AM, Richard Earnshaw wrote: > >> I understand (and agree with) this bit... >> >>> +(define_peephole2 >>> + [(set (reg:CC CC_REGNUM) >>> +(compare:CC (match_operand:SI 0 "register_operand" "") >>> +(match_opera

Re: [PATCH] ARM: Don't clobber CC reg when it is live after the peephole window

2013-06-10 Thread Meador Inge
On 06/06/2013 01:23 PM, Meador Inge wrote: > On 06/06/2013 08:11 AM, Richard Earnshaw wrote: > >> I understand (and agree with) this bit... >> >>> +(define_peephole2 >>> + [(set (reg:CC CC_REGNUM) >>> +(compare:CC (match_operand:SI 0 "register_operand" "") >>> +(match_operand:SI

Re: [PATCH] ARM: Don't clobber CC reg when it is live after the peephole window

2013-06-06 Thread Meador Inge
On 06/06/2013 08:11 AM, Richard Earnshaw wrote: > I understand (and agree with) this bit... > >> +(define_peephole2 >> + [(set (reg:CC CC_REGNUM) >> +(compare:CC (match_operand:SI 0 "register_operand" "") >> +(match_operand:SI 1 "arm_rhs_operand" ""))) >> + (cond_exec (ne (reg:

Re: [PATCH] ARM: Don't clobber CC reg when it is live after the peephole window

2013-06-06 Thread Richard Earnshaw
On 29/05/13 18:15, Meador Inge wrote: Hi All, This patch fixes a bug in one of the ARM peephole2 optimizations. The peephole2 optimization in question was changed to use the CC-updating form for all of the instructions produced by the peephole so that the encoding will be smaller when compiling

Re: [PATCH] ARM: Don't clobber CC reg when it is live after the peephole window

2013-06-05 Thread Meador Inge
Ping. On 05/29/2013 12:15 PM, Meador Inge wrote: > Hi All, > > This patch fixes a bug in one of the ARM peephole2 optimizations. The > peephole2 optimization in question was changed to use the CC-updating > form for all of the instructions produced by the peephole so that the > encoding will be

[PATCH] ARM: Don't clobber CC reg when it is live after the peephole window

2013-05-29 Thread Meador Inge
Hi All, This patch fixes a bug in one of the ARM peephole2 optimizations. The peephole2 optimization in question was changed to use the CC-updating form for all of the instructions produced by the peephole so that the encoding will be smaller when compiling for thumb [1]. However, I don't think