I tried that and it caused some regressions, so I choosed to do chain_to_prev_insn another time in add_branch_dependences. There could be some dependence between those two functions.
On Wed, Sep 11, 2013 at 2:58 AM, Alexander Monakov <amona...@ispras.ru> wrote: > > > On Tue, 10 Sep 2013, Wei Mi wrote: > >> Because deps_analyze_insn only analyzes data deps but no control deps. >> Control deps are included by add_branch_dependences. Without the >> chain_to_prev_insn in the end of add_branch_dependences, jmp will be >> control dependent on every previous insn in the same bb, and the cmp >> and jmp group could still be scheduled apart since they will not be >> put in ready list at the same time. > > Would calling add_branch_dependences before sched_analyze solve that, then? > > Alexander
