Re: [PATCH, PR66444] Handle -fipa-ra in reload_combine

2015-06-08 Thread Tom de Vries
On 08/06/15 17:31, Jakub Jelinek wrote: On Mon, Jun 08, 2015 at 02:04:12PM +0200, Tom de Vries wrote: >this patch fixes PR66444, a problem with -fipa-ra in reload_combine. > >The problem is that for the test-case, reload_combine combines these two >insns: Please work out with Vlad whether reloa

Re: [PATCH, PR66444] Handle -fipa-ra in reload_combine

2015-06-08 Thread Tom de Vries
On 08/06/15 17:31, Jakub Jelinek wrote: On Mon, Jun 08, 2015 at 02:04:12PM +0200, Tom de Vries wrote: this patch fixes PR66444, a problem with -fipa-ra in reload_combine. The problem is that for the test-case, reload_combine combines these two insns: Please work out with Vlad whether reload_c

Re: [PATCH, PR66444] Handle -fipa-ra in reload_combine

2015-06-08 Thread Jakub Jelinek
On Mon, Jun 08, 2015 at 02:04:12PM +0200, Tom de Vries wrote: > this patch fixes PR66444, a problem with -fipa-ra in reload_combine. > > The problem is that for the test-case, reload_combine combines these two > insns: Please work out with Vlad whether reload_cse_move2add doesn't need similar fix

[PATCH, PR66444] Handle -fipa-ra in reload_combine

2015-06-08 Thread Tom de Vries
Hi, this patch fixes PR66444, a problem with -fipa-ra in reload_combine. The problem is that for the test-case, reload_combine combines these two insns: ... (insn 13 12 14 2 (parallel [ (set (reg/v/f:DI 37 r8 [orig:92 xD.1858 ] [92]) (plus:DI (reg:DI 37 r8 [96])