Re: [PATCH] combine: Omit redundant AND in change_zero_ext.

2016-12-19 Thread Andreas Krebbel
On Wed, Dec 14, 2016 at 11:01:47AM +0100, Dominik Vogt wrote: > gcc/ChangeLog-change_zero_ext-2 > > * combine.c (change_zero_ext): Skip generation of redundant AND. Applied. Thanks! -Andreas-

Re: [PATCH] combine: Omit redundant AND in change_zero_ext.

2016-12-15 Thread Segher Boessenkool
On Thu, Dec 15, 2016 at 09:55:52AM +0100, Dominik Vogt wrote: > > > Bootstrapped and regression tested on s390x and s390. (Targets > > > with risbg-like instructions (Power, others?) may need some > > > tuning.) > > > > But, it is also possible I missed some. So please wait until I have > > test

Re: [PATCH] combine: Omit redundant AND in change_zero_ext.

2016-12-15 Thread Dominik Vogt
On Wed, Dec 14, 2016 at 01:32:48PM -0600, Segher Boessenkool wrote: > On Wed, Dec 14, 2016 at 11:01:47AM +0100, Dominik Vogt wrote: > > This is another micro-optimisation in change_zero_ext. If an > > > > (and (lshiftrt ... (N)) (M)) > > > > generated by change_zero_ext is equivalent to just >

Re: [PATCH] combine: Omit redundant AND in change_zero_ext.

2016-12-14 Thread Segher Boessenkool
On Wed, Dec 14, 2016 at 11:01:47AM +0100, Dominik Vogt wrote: > This is another micro-optimisation in change_zero_ext. If an > > (and (lshiftrt ... (N)) (M)) > > generated by change_zero_ext is equivalent to just > > (lshiftrt ... (N)) > > (because the AND constant selects the N rightmost