Re: [PATCH ARM]Extend thumb1_reorg to save more comparison instructions

2013-09-17 Thread Richard Earnshaw
On 17/09/13 03:16, bin.cheng wrote: > > >> -Original Message- >> From: Richard Earnshaw >> Sent: Thursday, September 12, 2013 11:24 PM >> To: Bin Cheng >> Cc: gcc-patches@gcc.gnu.org >> Subject: Re: [PATCH ARM]Extend thumb1_reorg to save more co

RE: [PATCH ARM]Extend thumb1_reorg to save more comparison instructions

2013-09-16 Thread bin.cheng
> -Original Message- > From: Richard Earnshaw > Sent: Thursday, September 12, 2013 11:24 PM > To: Bin Cheng > Cc: gcc-patches@gcc.gnu.org > Subject: Re: [PATCH ARM]Extend thumb1_reorg to save more comparison > instructions > > On 18/04/13 06:34, Bin Cheng wrot

Re: [PATCH ARM]Extend thumb1_reorg to save more comparison instructions

2013-09-12 Thread Richard Earnshaw
On 18/04/13 06:34, Bin Cheng wrote: > Hi, > Before thumb1_reorg, ARM backend uses peephole to save comparison > instructions when a flag setting move is found before branch instruction. > Since we are using thumb1_reog now, it can be extended to catch more > opportunities by searching flag setting

Re: [PATCH ARM]Extend thumb1_reorg to save more comparison instructions

2013-04-18 Thread Eric Botcazou
> I haven't looked at compare-elim, but probably not. Thumb1 doesn't use > cc[0] and doesn't expose flag setting instructions in a MI manner (since > add instructions clobber the flags). compare-elim is supposed to be designed for this kind of targets, but of course you need to split the instruc

RE: [PATCH ARM]Extend thumb1_reorg to save more comparison instructions

2013-04-18 Thread Bin Cheng
> -Original Message- > From: Richard Earnshaw > Sent: Thursday, April 18, 2013 5:25 PM > To: Steven Bosscher > Cc: Bin Cheng; gcc-patches@gcc.gnu.org > Subject: Re: [PATCH ARM]Extend thumb1_reorg to save more comparison > instructions > > On 18/04/13 09:20, St

Re: [PATCH ARM]Extend thumb1_reorg to save more comparison instructions

2013-04-18 Thread Richard Earnshaw
On 18/04/13 09:20, Steven Bosscher wrote: On Thu, Apr 18, 2013 at 7:34 AM, Bin Cheng wrote: Hi, Before thumb1_reorg, ARM backend uses peephole to save comparison instructions when a flag setting move is found before branch instruction. Since we are using thumb1_reog now, it can be extended to ca

Re: [PATCH ARM]Extend thumb1_reorg to save more comparison instructions

2013-04-18 Thread Steven Bosscher
On Thu, Apr 18, 2013 at 7:34 AM, Bin Cheng wrote: > Hi, > Before thumb1_reorg, ARM backend uses peephole to save comparison > instructions when a flag setting move is found before branch instruction. > Since we are using thumb1_reog now, it can be extended to catch more > opportunities by searching