Re: [PATCH, AArch64] Make zero_extends explicit for common SImode patterns

2012-12-17 Thread Richard Earnshaw
On 14/12/12 11:59, Ian Bolton wrote: Hi Richard, + "add\\t%w0, %w2, %w, xt" ^^^ %w1 Got spot. I guess that pattern hasn't fired yet then! I'll fix it. Now fixed in v3. I should have said that I am indeed running with REE enabled. It has some impact (about 70 furt

RE: [PATCH, AArch64] Make zero_extends explicit for common SImode patterns

2012-12-14 Thread Ian Bolton
> Hi Richard, > > > + "add\\t%w0, %w2, %w, xt" > > > > ^^^ %w1 > > Got spot. I guess that pattern hasn't fired yet then! I'll fix it. Now fixed in v3. > I should have said that I am indeed running with REE enabled. It has > some impact (about 70 further UXTW removed from

RE: [PATCH, AArch64] Make zero_extends explicit for common SImode patterns

2012-12-14 Thread Ian Bolton
Hi Richard, > + "add\\t%w0, %w2, %w, xt" > > ^^^ %w1 Got spot. I guess that pattern hasn't fired yet then! I'll fix it. > > This patch significantly reduces the number of redundant > > uxtw instructions seen in a variety of programs. > > > > (There are further patterns th

Re: [PATCH, AArch64] Make zero_extends explicit for common SImode patterns

2012-12-13 Thread Richard Henderson
On 12/13/2012 09:25 AM, Ian Bolton wrote: > This patch significantly reduces the number of redundant > uxtw instructions seen in a variety of programs. > > (There are further patterns that can be done, but I have them > in a separate patch that's still in development.) What do you get if you enab

Re: [PATCH, AArch64] Make zero_extends explicit for common SImode patterns

2012-12-13 Thread Richard Henderson
On 12/13/2012 09:25 AM, Ian Bolton wrote: > + "add\\t%w0, %w2, %w, xt" ^^^ %w1 r~