Re: patch to fix a wrong code generation with LRA when SDmode is used on ppc.

2014-08-09 Thread David Edelsohn
Hi, Vlad Thanks for the explanation. The patch is okay. Thanks, David On Fri, Aug 8, 2014 at 5:26 PM, Vladimir Makarov wrote: > On 2014-08-08, 2:53 PM, David Edelsohn wrote: >> >> Hi, Vlad >> >> Why does rs6000_emit_move need special support for SDmode with LRA and >> not with reload? > > >

Re: patch to fix a wrong code generation with LRA when SDmode is used on ppc.

2014-08-08 Thread Vladimir Makarov
On 2014-08-08, 2:53 PM, David Edelsohn wrote: Hi, Vlad Why does rs6000_emit_move need special support for SDmode with LRA and not with reload? rs6000_emit_move has also special SDmode support for *reload too*. It is not implemented generically in reload as you wrote. Please, look at the c

Re: patch to fix a wrong code generation with LRA when SDmode is used on ppc.

2014-08-08 Thread David Edelsohn
Hi, Vlad Why does rs6000_emit_move need special support for SDmode with LRA and not with reload? In other words, why is this a specific fix for rs6000 instead of a general improvement for reload? SDmode has some bizarre restrictions on PPC, but one of the historical advantages of GCC was reload'

patch to fix a wrong code generation with LRA when SDmode is used on ppc.

2014-08-07 Thread Vladimir Makarov
Hi, Mike. Here is the patch which solves LRA SDmode problem which we discussed on IRC on Friday. The patch was successfully tested and bootstrapped on ppc64 with -mlra switched on. David, is it ok to commit it to the trunk? 2014-08-07 Vladimir Makarov * config/rs6000/rs6000.c (rs600