On 06/02/2011 01:29 PM, Alexander Monakov wrote: > Bernd, > > The problem is INSN_COND should be reset when initializing a new deps > structure, otherwise instructions may get stale conditions from other > previously analyzed instructions. Presuming that sd_init_insn is the > proper place for that, I'll test the following patch. > > > 2011-06-02 Alexander Monakov <amona...@ispras.ru> > > * sched-deps.c (sd_init_insn): Initialize INSN_COND. > * sel-sched.c (move_op): Use correct type for 'res'. Verify that > code_motion_path_driver returned 0 or 1.
Ok. Although I wonder how sel-sched can end up reusing an entry in h_d_i_d? How does it use this machinery? If it's not doing a normal forward scan as in sched_analyze, the INSN_COND mechanism may break in other ways. Bernd