Re: [PATCH 6/7] ira.c use DF infrastructure for combine_and_move_insns

2016-03-22 Thread Alan Modra
On Tue, Mar 22, 2016 at 05:29:08PM +0100, Bernd Schmidt wrote: > On 03/21/2016 02:42 AM, Alan Modra wrote: > > * ira.c (combine_and_move_insns): Rather than scanning insns, > > use DF infrastucture to find use and def insns. > > > >- remove_death (regno, insn); > > This ca

Re: [PATCH 6/7] ira.c use DF infrastructure for combine_and_move_insns

2016-03-22 Thread Bernd Schmidt
On 03/21/2016 02:42 AM, Alan Modra wrote: * ira.c (combine_and_move_insns): Rather than scanning insns, use DF infrastucture to find use and def insns. - remove_death (regno, insn); This call appears to have gone missing. Is that intentional? Other than tha

[PATCH 6/7] ira.c use DF infrastructure for combine_and_move_insns

2016-03-20 Thread Alan Modra
This patch actually improves generated code, because REG_DEAD notes used by the old insn scan are not always present. On x86_64, see gcc/wide-int-print.o:print_hex for an example of a function that is smaller and uses one less callee saved reg. * ira.c (combine_and_move_insns): Rather tha