Re: Incomplete instatitiation of virtual registers

2005-05-05 Thread Martin Koegler
On Wed, May 04, 2005 at 01:02:18AM -0700, Richard Henderson wrote: > On Wed, May 04, 2005 at 09:50:49AM +0200, Martin Koegler wrote: > > For that instruction, instantiate_virtual_regs_in_insn > > enters if(set), then if (GET_CODE (SET_SRC (set)) == PLUS > > is entered, where if (safe_insn_predicat

Re: Incomplete instatitiation of virtual registers

2005-05-04 Thread Martin Koegler
On Wed, May 04, 2005 at 01:02:18AM -0700, Richard Henderson wrote: > On Wed, May 04, 2005 at 09:50:49AM +0200, Martin Koegler wrote: > > For that instruction, instantiate_virtual_regs_in_insn > > enters if(set), then if (GET_CODE (SET_SRC (set)) == PLUS > > is entered, where if (safe_insn_predicat

Re: Incomplete instatitiation of virtual registers

2005-05-04 Thread Richard Henderson
On Wed, May 04, 2005 at 09:50:49AM +0200, Martin Koegler wrote: > For that instruction, instantiate_virtual_regs_in_insn > enters if(set), then if (GET_CODE (SET_SRC (set)) == PLUS > is entered, where if (safe_insn_predicate (insn_code, 1, new) is entered. > It then jumps to verify, without changi

Re: Incomplete instatitiation of virtual registers

2005-05-04 Thread Martin Koegler
On Tue, May 03, 2005 at 05:44:47PM -0700, James E Wilson wrote: > Martin Koegler wrote: > >I notice, that your last change in function.c forgets virtual > >registers in the RTL in some conditions. In older version (the last I used > >was 20050412), > >this has not happend. > > Patches should go t

Re: Incomplete instatitiation of virtual registers

2005-05-03 Thread Richard Henderson
On Tue, May 03, 2005 at 05:44:47PM -0700, James E Wilson wrote: > Recursively calling instantiate_virtual_regs_in_insn does not look > right. Indeed it is not. I'd like to see the define_insn for {addhi3}. I'm a bit confused as to how I could have missed iterating over what appears like it ough

Re: Incomplete instatitiation of virtual registers

2005-05-03 Thread James E Wilson
Martin Koegler wrote: I notice, that your last change in function.c forgets virtual registers in the RTL in some conditions. In older version (the last I used was 20050412), this has not happend. Patches should go to gcc-patches instead of the gcc list. If you want us to continue accepting patches

Incomplete instatitiation of virtual registers

2005-05-01 Thread Martin Koegler
I notice, that your last change in function.c forgets virtual registers in the RTL in some conditions. In older version (the last I used was 20050412), this has not happend. In 01.sibling, I have the instruction: (insn 10 8 12 1 (set (mem/f/i:HI (plus:HI (reg/f:HI 23 virtual-stack-vars)