Re: [PATCH, rs6000] Improve TImode add/sub

2014-04-29 Thread David Edelsohn
On Mon, Apr 28, 2014 at 3:33 PM, Pat Haugen wrote: > On 04/16/2014 10:27 PM, David Edelsohn wrote: >>> >>> >Updated patch with above comments incorporated. Bootstrap/regtest on >>> > BE/LE >>> >with no new regressions. Ok for trunk? >> >> 2014-04-08 Pat Haugen >> >> * config/rs6000/rs600

Re: [PATCH, rs6000] Improve TImode add/sub

2014-04-28 Thread David Edelsohn
On Mon, Apr 28, 2014 at 3:33 PM, Pat Haugen wrote: > On 04/16/2014 10:27 PM, David Edelsohn wrote: >>> >>> >Updated patch with above comments incorporated. Bootstrap/regtest on >>> > BE/LE >>> >with no new regressions. Ok for trunk? >> >> 2014-04-08 Pat Haugen >> >> * config/rs6000/rs600

Re: [PATCH, rs6000] Improve TImode add/sub

2014-04-28 Thread Pat Haugen
On 04/16/2014 10:27 PM, David Edelsohn wrote: >Updated patch with above comments incorporated. Bootstrap/regtest on BE/LE >with no new regressions. Ok for trunk? 2014-04-08 Pat Haugen * config/rs6000/rs6000.md (addti3, subti3): New. gcc/testsuite: * gcc.target/powerpc/ti_mat

Re: [PATCH, rs6000] Improve TImode add/sub

2014-04-16 Thread David Edelsohn
On Wed, Apr 16, 2014 at 4:51 PM, Pat Haugen wrote: > Updated patch with above comments incorporated. Bootstrap/regtest on BE/LE > with no new regressions. Ok for trunk? 2014-04-08 Pat Haugen * config/rs6000/rs6000.md (addti3, subti3): New. gcc/testsuite: * gcc.target/powerpc

Re: [PATCH, rs6000] Improve TImode add/sub

2014-04-16 Thread Pat Haugen
On 04/08/2014 09:56 PM, seg...@kernel.crashing.org wrote: +/* { dg-do compile { target { powerpc*-*-* && lp64 } } } */ +/* { dg-skip-if "" { powerpc*-*-darwin* } { "*" } { "" } } */ Please leave out the default arguments. Why does this need skipping on Darwin? +;; Define the TImode operations

Re: [PATCH, rs6000] Improve TImode add/sub

2014-04-08 Thread segher
Hello, On Tue, Apr 08, 2014 at 07:06:58PM -0500, Pat Haugen wrote: > The following patch improves the code generated for TImode add/sub so > that we now generate a 2 insn sequence which makes use of the carry bit. > +/* { dg-do compile { target { powerpc*-*-* && lp64 } } } */ > +/* { dg-skip-if

[PATCH, rs6000] Improve TImode add/sub

2014-04-08 Thread Pat Haugen
The following patch improves the code generated for TImode add/sub so that we now generate a 2 insn sequence which makes use of the carry bit. Bootstrap/regtest (on both BE/LE) with no new failures. Ok for trunk and 4.8? -Pat 2014-04-08 Pat Haugen * config/rs6000/rs6000.md (addti