Re: [RS6000] bswapdi2 pattern, reload and lra

2015-03-15 Thread Alan Modra
On Sun, Mar 15, 2015 at 10:51:02PM -0400, David Edelsohn wrote: > On Tue, Dec 17, 2013 at 6:50 AM, Alan Modra wrote: PR target/61350 > > gcc/ > > * config/rs6000/rs6000.md (bswapdi2): Remove one scratch reg. > > Modify Z->r bswapdi splitter to use dest in place of scratch.

Re: [RS6000] bswapdi2 pattern, reload and lra

2015-03-15 Thread David Edelsohn
On Tue, Dec 17, 2013 at 6:50 AM, Alan Modra wrote: > This patch is aimed at fixing test failures introduced by my > 2013-12-07 change to bswapdi2_32bit: > FAIL: gcc.target/powerpc/pr53199.c scan-assembler-times lwbrx 6 > FAIL: gcc.target/powerpc/pr53199.c scan-assembler-times stwbrx 6 > > The 2013

Re: [RS6000] bswapdi2 pattern, reload and lra

2015-03-12 Thread Alan Modra
On Wed, Dec 18, 2013 at 09:53:38AM -0500, David Edelsohn wrote: https://gcc.gnu.org/ml/gcc-patches/2013-12/msg01599.html > Why change the code from swapping the words at the initial > change_address() to swapping the words in the call to gen_bswapsi2()? Sorry for dropping this on the floor for so

Re: [RS6000] bswapdi2 pattern, reload and lra

2013-12-18 Thread David Edelsohn
On Tue, Dec 17, 2013 at 6:50 AM, Alan Modra wrote: > Bootstrapped and regression tested powerpc64-linux. Output of > pr53199.c inspected for sanity with -mcpu=power{6,7} -m{32,64} and > {-mlra,}. OK to apply? > > gcc/ > * config/rs6000/rs6000.md (bswapdi2): Remove one scratch reg. >

[RS6000] bswapdi2 pattern, reload and lra

2013-12-17 Thread Alan Modra
This patch is aimed at fixing test failures introduced by my 2013-12-07 change to bswapdi2_32bit: FAIL: gcc.target/powerpc/pr53199.c scan-assembler-times lwbrx 6 FAIL: gcc.target/powerpc/pr53199.c scan-assembler-times stwbrx 6 The 2013-12-07 change was necessary to make -m32 -mlra generate good co