Re: Patch ping (Re: [PATCH] Fix PR81175, make gather builtins pure)

2017-07-04 Thread Richard Biener
On Tue, 4 Jul 2017, Jakub Jelinek wrote: > Hi! > > On Tue, Jun 27, 2017 at 12:27:25PM +0200, Jakub Jelinek wrote: > > Fixed thusly, ok for trunk? Perhaps we should add another testcase to check > > similarly gatherpf builtin without the lhs, but we'd need different options. > > I'd like to ping

Re: Patch ping (Re: [PATCH] Fix PR81175, make gather builtins pure)

2017-07-04 Thread Uros Bizjak
On Tue, Jul 4, 2017 at 10:35 AM, Jakub Jelinek wrote: > Hi! > > On Tue, Jun 27, 2017 at 12:27:25PM +0200, Jakub Jelinek wrote: >> Fixed thusly, ok for trunk? Perhaps we should add another testcase to check >> similarly gatherpf builtin without the lhs, but we'd need different options. > > I'd lik

Patch ping (Re: [PATCH] Fix PR81175, make gather builtins pure)

2017-07-04 Thread Jakub Jelinek
Hi! On Tue, Jun 27, 2017 at 12:27:25PM +0200, Jakub Jelinek wrote: > Fixed thusly, ok for trunk? Perhaps we should add another testcase to check > similarly gatherpf builtin without the lhs, but we'd need different options. I'd like to ping this patch, ok for trunk? > 2017-06-27 Jakub Jelinek

Re: [PATCH] Fix PR81175, make gather builtins pure

2017-06-27 Thread Richard Biener
On Tue, 27 Jun 2017, Jakub Jelinek wrote: > On Fri, Jun 23, 2017 at 02:54:35PM +0200, Richard Biener wrote: > > 2017-06-23 Richard Biener > > > > PR target/81175 > > * config/i386/i386.c (struct builtin_isa): Add pure_p member. > > (def_builtin2): Initialize pure_p. > > (ix86_a

Re: [PATCH] Fix PR81175, make gather builtins pure

2017-06-27 Thread Jakub Jelinek
On Tue, Jun 27, 2017 at 12:08:59PM +0200, Uros Bizjak wrote: > > Ok for trunk if it passes bootstrap/regtest? > > > > 2017-06-27 Jakub Jelinek > > > > PR target/81175 > > * config/i386/i386.c (ix86_init_mmx_sse_builtins): Use def_builtin > > rather than def_builtin_pure f

Re: [PATCH] Fix PR81175, make gather builtins pure

2017-06-27 Thread Uros Bizjak
On Tue, Jun 27, 2017 at 12:02 PM, Jakub Jelinek wrote: > On Fri, Jun 23, 2017 at 02:54:35PM +0200, Richard Biener wrote: >> 2017-06-23 Richard Biener >> >> PR target/81175 >> * config/i386/i386.c (struct builtin_isa): Add pure_p member. >> (def_builtin2): Initialize pure_p. >>

Re: [PATCH] Fix PR81175, make gather builtins pure

2017-06-27 Thread Jakub Jelinek
On Fri, Jun 23, 2017 at 02:54:35PM +0200, Richard Biener wrote: > 2017-06-23 Richard Biener > > PR target/81175 > * config/i386/i386.c (struct builtin_isa): Add pure_p member. > (def_builtin2): Initialize pure_p. > (ix86_add_new_builtins): Honor pure_p. > (def_buil

Re: [PATCH] Fix PR81175, make gather builtins pure

2017-06-26 Thread Uros Bizjak
On Fri, Jun 23, 2017 at 3:22 PM, Richard Biener wrote: > On Fri, 23 Jun 2017, Marc Glisse wrote: > >> On Fri, 23 Jun 2017, Richard Biener wrote: >> >> > The vectorizer is confused about the spurious VDEFs that are caused >> > by gather vectorization so the following avoids them by making the >> >

Re: [PATCH] Fix PR81175, make gather builtins pure

2017-06-23 Thread Jakub Jelinek
On Fri, Jun 23, 2017 at 03:22:13PM +0200, Richard Biener wrote: > On Fri, 23 Jun 2017, Marc Glisse wrote: > > > On Fri, 23 Jun 2017, Richard Biener wrote: > > > > > The vectorizer is confused about the spurious VDEFs that are caused > > > by gather vectorization so the following avoids them by ma

Re: [PATCH] Fix PR81175, make gather builtins pure

2017-06-23 Thread Richard Biener
On Fri, 23 Jun 2017, Marc Glisse wrote: > On Fri, 23 Jun 2017, Richard Biener wrote: > > > The vectorizer is confused about the spurious VDEFs that are caused > > by gather vectorization so the following avoids them by making the > > builtins pure appropriately. > > > > Bootstrap / regtest pendi

Re: [PATCH] Fix PR81175, make gather builtins pure

2017-06-23 Thread Marc Glisse
On Fri, 23 Jun 2017, Richard Biener wrote: The vectorizer is confused about the spurious VDEFs that are caused by gather vectorization so the following avoids them by making the builtins pure appropriately. Bootstrap / regtest pending on x86_64-unknown-linux-gnu, ok for trunk and branch? Thank

[PATCH] Fix PR81175, make gather builtins pure

2017-06-23 Thread Richard Biener
The vectorizer is confused about the spurious VDEFs that are caused by gather vectorization so the following avoids them by making the builtins pure appropriately. Bootstrap / regtest pending on x86_64-unknown-linux-gnu, ok for trunk and branch? Thanks, Richard. 2017-06-23 Richard Biener