Re: [PATCH] match_asm_constraints: Use copy_rtx where needed (PR88001)

2018-12-15 Thread Segher Boessenkool
On Thu, Dec 13, 2018 at 11:09:14AM -0700, Jeff Law wrote: > On 12/12/18 1:22 PM, Segher Boessenkool wrote: > > The new insn here (temporarily) illegally shares RTL. This fixes it. > > > > Tested with an ARC cross, and regstrapped on powerpc64-linux {-m32,-m64}. > > Is this okay for trunk? > > >

Re: [PATCH] match_asm_constraints: Use copy_rtx where needed (PR88001)

2018-12-14 Thread Segher Boessenkool
On Thu, Dec 13, 2018 at 11:09:14AM -0700, Jeff Law wrote: > On 12/12/18 1:22 PM, Segher Boessenkool wrote: > > The new insn here (temporarily) illegally shares RTL. This fixes it. > > > > Tested with an ARC cross, and regstrapped on powerpc64-linux {-m32,-m64}. > > Is this okay for trunk? > > >

Re: [PATCH] match_asm_constraints: Use copy_rtx where needed (PR88001)

2018-12-13 Thread Jeff Law
On 12/12/18 1:22 PM, Segher Boessenkool wrote: > The new insn here (temporarily) illegally shares RTL. This fixes it. > > Tested with an ARC cross, and regstrapped on powerpc64-linux {-m32,-m64}. > Is this okay for trunk? > > > Segher > > > 2018-12-12 Segher Boessenkool > > PR rtl-o

[PATCH] match_asm_constraints: Use copy_rtx where needed (PR88001)

2018-12-12 Thread Segher Boessenkool
The new insn here (temporarily) illegally shares RTL. This fixes it. Tested with an ARC cross, and regstrapped on powerpc64-linux {-m32,-m64}. Is this okay for trunk? Segher 2018-12-12 Segher Boessenkool PR rtl-optimization/88001 * function.c (match_asm_constraints_1): Don