Re: [PATCH 1/6] rs6000: Remove "O" alternative from lshrsi3

2014-06-22 Thread David Edelsohn
On Sat, Jun 21, 2014 at 10:46 PM, Segher Boessenkool wrote: > Nothing will ever generate RTL matching this alternative. Maybe long > ago this was needed, but not anymore. > > Bootstrapped and tested on powerpc64-linux, {-m64,-m64/-mtune=power8, > -m32,-m32/-mpowerpc64}, no regressions. Okay to a

Re: [PATCH 1/6] rs6000: Remove "O" alternative from lshrsi3

2014-06-22 Thread Segher Boessenkool
> > Nothing will ever generate RTL matching this alternative. Maybe long > > ago this was needed, but not anymore. > > Can you provide some reference for that assertion? > > At some point GCC created shifts with zero shift count. It probably > does not, but does a generic optimization prevent th

Re: [PATCH 1/6] rs6000: Remove "O" alternative from lshrsi3

2014-06-22 Thread David Edelsohn
On Sat, Jun 21, 2014 at 10:46 PM, Segher Boessenkool wrote: > Nothing will ever generate RTL matching this alternative. Maybe long > ago this was needed, but not anymore. Can you provide some reference for that assertion? At some point GCC created shifts with zero shift count. It probably does

[PATCH 1/6] rs6000: Remove "O" alternative from lshrsi3

2014-06-21 Thread Segher Boessenkool
Nothing will ever generate RTL matching this alternative. Maybe long ago this was needed, but not anymore. Bootstrapped and tested on powerpc64-linux, {-m64,-m64/-mtune=power8, -m32,-m32/-mpowerpc64}, no regressions. Okay to apply? Segher 2014-06-21 Segher Boessenkool gcc/ * conf