Re: [PATCH v2, middle-end]: Introduce memory_blockage named insn pattern

2017-10-14 Thread David Edelsohn
On Sat, Oct 14, 2017 at 1:29 PM, Uros Bizjak wrote: > On Sat, Oct 14, 2017 at 5:44 PM, David Edelsohn wrote: >> This patch has broken bootstrap on AIX and possibly powerpc64-linux. >> Was this patch tested on any architecture other than x86? > > No. > >> /nasfarm/edelsohn/src/src/libgcc/emutls.c:

Re: [PATCH v2, middle-end]: Introduce memory_blockage named insn pattern

2017-10-14 Thread Uros Bizjak
On Sat, Oct 14, 2017 at 5:44 PM, David Edelsohn wrote: > This patch has broken bootstrap on AIX and possibly powerpc64-linux. > Was this patch tested on any architecture other than x86? No. > /nasfarm/edelsohn/src/src/libgcc/emutls.c: In function '__emutls_get_address': > /nasfarm/edelsohn/src/s

Re: [PATCH v2, middle-end]: Introduce memory_blockage named insn pattern

2017-10-14 Thread David Edelsohn
On Sat, Oct 14, 2017 at 11:44 AM, David Edelsohn wrote: > This patch has broken bootstrap on AIX and possibly powerpc64-linux. > Was this patch tested on any architecture other than x86? > > /nasfarm/edelsohn/src/src/libgcc/emutls.c: In function '__emutls_get_address': > /nasfarm/edelsohn/src/src/

Re: [PATCH v2, middle-end]: Introduce memory_blockage named insn pattern

2017-10-14 Thread David Edelsohn
This patch has broken bootstrap on AIX and possibly powerpc64-linux. Was this patch tested on any architecture other than x86? /nasfarm/edelsohn/src/src/libgcc/emutls.c: In function '__emutls_get_address': /nasfarm/edelsohn/src/src/libgcc/emutls.c:139:11: internal compiler error: in invalid_void,

Re: [PATCH v2, middle-end]: Introduce memory_blockage named insn pattern

2017-10-14 Thread Uros Bizjak
On Sat, Oct 14, 2017 at 12:16 PM, Andrew Pinski wrote: > On Mon, Sep 18, 2017 at 2:06 PM, Uros Bizjak wrote: >> On Tue, Sep 5, 2017 at 3:50 PM, Uros Bizjak wrote: >>> Revised patch, incorporates fixes from Alexander's review comments. >>> >>> I removed some implementation details from Alexander'

Re: [PATCH v2, middle-end]: Introduce memory_blockage named insn pattern

2017-10-14 Thread Christophe Lyon
On 14 October 2017 at 12:16, Andrew Pinski wrote: > On Mon, Sep 18, 2017 at 2:06 PM, Uros Bizjak wrote: >> On Tue, Sep 5, 2017 at 3:50 PM, Uros Bizjak wrote: >>> Revised patch, incorporates fixes from Alexander's review comments. >>> >>> I removed some implementation details from Alexander's des

Re: [PATCH v2, middle-end]: Introduce memory_blockage named insn pattern

2017-10-14 Thread Andrew Pinski
On Mon, Sep 18, 2017 at 2:06 PM, Uros Bizjak wrote: > On Tue, Sep 5, 2017 at 3:50 PM, Uros Bizjak wrote: >> Revised patch, incorporates fixes from Alexander's review comments. >> >> I removed some implementation details from Alexander's description of >> memory_blockage named pattern. >> >> >> 20

Re: [PATCH v2, middle-end]: Introduce memory_blockage named insn pattern

2017-10-13 Thread Jeff Law
On 10/13/2017 12:27 PM, Uros Bizjak wrote: > On Fri, Oct 13, 2017 at 7:30 PM, Jeff Law wrote: >> On 09/05/2017 07:50 AM, Uros Bizjak wrote: >>> Revised patch, incorporates fixes from Alexander's review comments. >>> >>> I removed some implementation details from Alexander's description of >>> memo

Re: [PATCH v2, middle-end]: Introduce memory_blockage named insn pattern

2017-10-13 Thread Uros Bizjak
On Fri, Oct 13, 2017 at 7:30 PM, Jeff Law wrote: > On 09/05/2017 07:50 AM, Uros Bizjak wrote: >> Revised patch, incorporates fixes from Alexander's review comments. >> >> I removed some implementation details from Alexander's description of >> memory_blockage named pattern. >> >> >> 2017-09-05 Ur

Re: [PATCH v2, middle-end]: Introduce memory_blockage named insn pattern

2017-10-13 Thread Jeff Law
On 09/05/2017 07:50 AM, Uros Bizjak wrote: > Revised patch, incorporates fixes from Alexander's review comments. > > I removed some implementation details from Alexander's description of > memory_blockage named pattern. > > > 2017-09-05 Uros Bizjak > > * target-insns.def: Add memory_bloc

Re: [PATCH v2, middle-end]: Introduce memory_blockage named insn pattern

2017-09-18 Thread Uros Bizjak
On Tue, Sep 5, 2017 at 3:50 PM, Uros Bizjak wrote: > Revised patch, incorporates fixes from Alexander's review comments. > > I removed some implementation details from Alexander's description of > memory_blockage named pattern. > > > 2017-09-05 Uros Bizjak > > * target-insns.def: Add memory

Re: [PATCH v2, middle-end]: Introduce memory_blockage named insn pattern

2017-09-05 Thread Alexander Monakov
On Tue, 5 Sep 2017, Uros Bizjak wrote: > Revised patch, incorporates fixes from Alexander's review comments. > > I removed some implementation details from Alexander's description of > memory_blockage named pattern. Well, to me it wasn't really obvious why a named pattern was needed in the first

[PATCH v2, middle-end]: Introduce memory_blockage named insn pattern

2017-09-05 Thread Uros Bizjak
Revised patch, incorporates fixes from Alexander's review comments. I removed some implementation details from Alexander's description of memory_blockage named pattern. 2017-09-05 Uros Bizjak * target-insns.def: Add memory_blockage. * optabs.c (expand_memory_blockage): New function.