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'