Re: New assert in haifa-sched.c

2008-09-16 Thread Eric Botcazou
> I don't see the logic here. It is certainly not that hard to fix the > code to make this assertion trivial, simple loop through all insns in > sched_init() will do it. Yes, but apparently nobody is willing to do that at the moment so we'll need to kludge until then and add calls to recog_memoi

Re: New assert in haifa-sched.c

2008-09-16 Thread Adam Nemet
Eric Botcazou writes: > > Yes, the assert is really checking exactly that. Several pieces of > > haifa-sched.c assume that the instruction has been recognized during > > scheduler initialization to speed up checking if instruction is normal > > or some kind of use/clobber/asm. > > What happens if

Re: New assert in haifa-sched.c

2008-09-16 Thread Maxim Kuvyrkov
Eric Botcazou wrote: Yes, the assert is really checking exactly that. Several pieces of haifa-sched.c assume that the instruction has been recognized during scheduler initialization to speed up checking if instruction is normal or some kind of use/clobber/asm. What happens if an instruction ha

Re: New assert in haifa-sched.c

2008-09-16 Thread Eric Botcazou
> Yes, the assert is really checking exactly that. Several pieces of > haifa-sched.c assume that the instruction has been recognized during > scheduler initialization to speed up checking if instruction is normal > or some kind of use/clobber/asm. What happens if an instruction hasn't been recogn

Re: New assert in haifa-sched.c

2008-09-07 Thread Maxim Kuvyrkov
Adam Nemet wrote: Maxim Kuvyrkov writes: I'm not 100% sure about current state of things, considering recent merge of sel-sched, but before that it was: set_priorities() -> priority() -> dep_cost() -> recog_memoized(). I don't think that was the case for all insns even before the patch. The

Re: New assert in haifa-sched.c

2008-09-05 Thread Adam Nemet
Maxim Kuvyrkov writes: > I'm not 100% sure about current state of things, considering recent > merge of sel-sched, but before that it was: > > set_priorities() -> priority() -> dep_cost() -> recog_memoized(). I don't think that was the case for all insns even before the patch. The only new thin

Re: New assert in haifa-sched.c

2008-09-05 Thread Maxim Kuvyrkov
Adam Nemet wrote: Maxim Kuvyrkov writes: Yes, the assert is really checking exactly that. Several pieces of haifa-sched.c assume that the instruction has been recognized during scheduler initialization to speed up checking if instruction is normal or some kind of use/clobber/asm. Thanks for

Re: New assert in haifa-sched.c

2008-09-05 Thread Adam Nemet
Maxim Kuvyrkov writes: > Yes, the assert is really checking exactly that. Several pieces of > haifa-sched.c assume that the instruction has been recognized during > scheduler initialization to speed up checking if instruction is normal > or some kind of use/clobber/asm. Thanks for the info but

Re: New assert in haifa-sched.c

2008-09-05 Thread Maxim Kuvyrkov
Adam Nemet wrote: haifa-sched.c: 2302/* Let the target filter the search space. */ 2303for (i = 1; i < ready->n_ready; i++) 2304 if (!ready_try[i]) 2305{ 2306 insn = ready_element (ready, i); 2307 2308 gcc_assert (INS