Re: [PATCH] combine: Tweak change_zero_ext

2016-11-30 Thread Kyrill Tkachov
On 30/11/16 10:02, Bin.Cheng wrote: On Tue, Nov 29, 2016 at 9:14 PM, Christophe Lyon wrote: On 29 November 2016 at 20:38, Uros Bizjak wrote: 2016-11-26 Segher Boessenkool * combine.c (change_zero_ext): Also handle extends from a subreg to a mode bigger than that of the operand of the sub

Re: [PATCH] combine: Tweak change_zero_ext

2016-11-30 Thread Bin.Cheng
On Tue, Nov 29, 2016 at 9:14 PM, Christophe Lyon wrote: > On 29 November 2016 at 20:38, Uros Bizjak wrote: >>> 2016-11-26 Segher Boessenkool >>> >>> * combine.c (change_zero_ext): Also handle extends from a subreg >>> to a mode bigger than that of the operand of the subreg. >> >> This patch in

Re: [PATCH] combine: Tweak change_zero_ext

2016-11-29 Thread Christophe Lyon
On 29 November 2016 at 20:38, Uros Bizjak wrote: >> 2016-11-26 Segher Boessenkool >> >> * combine.c (change_zero_ext): Also handle extends from a subreg >> to a mode bigger than that of the operand of the subreg. > > This patch introduced: > > FAIL: gcc.target/i386/pr44578.c (internal compiler

Re: [PATCH] combine: Tweak change_zero_ext

2016-11-29 Thread Uros Bizjak
> 2016-11-26 Segher Boessenkool > > * combine.c (change_zero_ext): Also handle extends from a subreg > to a mode bigger than that of the operand of the subreg. This patch introduced: FAIL: gcc.target/i386/pr44578.c (internal compiler error) on i686 (or x86_64 32bit multi-lib). ./cc1 -O2 -mtu

[PATCH] combine: Tweak change_zero_ext

2016-11-26 Thread Segher Boessenkool
change_zero_ext handles (zero_extend:M1 (subreg:M2 (reg:M1) ...)) already; this patch extends it to also deal with any (zero_extend:M1 (subreg:M2 (reg:M3) ...)) where the subreg is not paradoxical. Tested on powerpc64-linux. This is needed for some of the rlwimi testcases to be optimised properly