Re: [PATCH 2/5] combine: handle I2 a parallel of two SETs

2014-11-26 Thread Segher Boessenkool
On Sat, Nov 15, 2014 at 06:59:23AM -0600, Segher Boessenkool wrote: > On Fri, Nov 14, 2014 at 08:35:48PM +0100, Bernd Schmidt wrote: > > On 11/14/2014 08:19 PM, Segher Boessenkool wrote: > > >+ /* If I2 is a PARALLEL of two SETs of REGs (and perhaps some CLOBBERs), > > >+ make those two SETs s

Re: [PATCH 2/5] combine: handle I2 a parallel of two SETs

2014-11-25 Thread Segher Boessenkool
On Tue, Nov 25, 2014 at 11:35:14AM -0700, Jeff Law wrote: > On 11/14/14 12:19, Segher Boessenkool wrote: > >If I2 is a PARALLEL of two SETs, split it into two instructions, I1 > >and I2. If there already was an I1, rename it to I0. If there > >already was an I0, don't do anything. > > > >This sur

Re: [PATCH 2/5] combine: handle I2 a parallel of two SETs

2014-11-25 Thread Jeff Law
On 11/14/14 12:19, Segher Boessenkool wrote: If I2 is a PARALLEL of two SETs, split it into two instructions, I1 and I2. If there already was an I1, rename it to I0. If there already was an I0, don't do anything. This surprisingly simple patch is enough to let combine handle such PARALLELs pro

Re: [PATCH 2/5] combine: handle I2 a parallel of two SETs

2014-11-15 Thread Segher Boessenkool
On Fri, Nov 14, 2014 at 08:35:48PM +0100, Bernd Schmidt wrote: > On 11/14/2014 08:19 PM, Segher Boessenkool wrote: > >+ /* If I2 is a PARALLEL of two SETs of REGs (and perhaps some CLOBBERs), > >+ make those two SETs separate I1 and I2 insns, and make an I0 that is > >+ the original I1. *

Re: [PATCH 2/5] combine: handle I2 a parallel of two SETs

2014-11-14 Thread Bernd Schmidt
On 11/14/2014 08:19 PM, Segher Boessenkool wrote: + /* If I2 is a PARALLEL of two SETs of REGs (and perhaps some CLOBBERs), + make those two SETs separate I1 and I2 insns, and make an I0 that is + the original I1. */ + if (i0 == 0 + && GET_CODE (PATTERN (i2)) == PARALLEL + &&

[PATCH 2/5] combine: handle I2 a parallel of two SETs

2014-11-14 Thread Segher Boessenkool
If I2 is a PARALLEL of two SETs, split it into two instructions, I1 and I2. If there already was an I1, rename it to I0. If there already was an I0, don't do anything. This surprisingly simple patch is enough to let combine handle such PARALLELs properly. 2014-11-14 Segher Boessenkool gcc/