Re: [RFC PATCH] Improve switchconv optimization (PR tree-optimization/79472)

2017-02-15 Thread Jeff Law
On 02/15/2017 04:51 AM, Jakub Jelinek wrote: On Wed, Feb 15, 2017 at 12:46:44PM +0100, Richard Biener wrote: Possibly, but for GCC 8. To both this switchconv patch and the potential improvement for loading from const arrays (can create an enhancement PR for that), or just the latter? Both I

Re: [RFC PATCH] Improve switchconv optimization (PR tree-optimization/79472)

2017-02-15 Thread Jakub Jelinek
On Wed, Feb 15, 2017 at 12:46:44PM +0100, Richard Biener wrote: > >> Possibly, but for GCC 8. > > > > To both this switchconv patch and the potential improvement for loading > > from const arrays (can create an enhancement PR for that), or just the > > latter? > > Both I think - the patch is pret

Re: [RFC PATCH] Improve switchconv optimization (PR tree-optimization/79472)

2017-02-15 Thread Richard Biener
On 15/02/17 08:17, Jakub Jelinek wrote: > On Wed, Feb 15, 2017 at 08:06:16AM +0100, Richard Biener wrote: >> On February 14, 2017 9:04:45 PM GMT+01:00, Jakub Jelinek >> wrote: >>> Hi! >>> >>> The following patch is an attempt to fix a regression where we no >>> longer >>> switch convert one switc

Re: [RFC PATCH] Improve switchconv optimization (PR tree-optimization/79472)

2017-02-14 Thread Jakub Jelinek
On Wed, Feb 15, 2017 at 08:06:16AM +0100, Richard Biener wrote: > On February 14, 2017 9:04:45 PM GMT+01:00, Jakub Jelinek > wrote: > >Hi! > > > >The following patch is an attempt to fix a regression where we no > >longer > >switch convert one switch because earlier optimizations turn it into > >

Re: [RFC PATCH] Improve switchconv optimization (PR tree-optimization/79472)

2017-02-14 Thread Richard Biener
On February 14, 2017 9:04:45 PM GMT+01:00, Jakub Jelinek wrote: >Hi! > >The following patch is an attempt to fix a regression where we no >longer >switch convert one switch because earlier optimizations turn it into >unsupported shape. Is that because of early threading? >The patch contains two