On Wed, 11 Sep 2013, Wei Mi wrote: > I agree with you that explicit handling in sched-deps.c for this > feature looks not good. So I move it to sched_init (Instead of > ix86_sched_init_global because ix86_sched_init_global is used to > install scheduling hooks), and then it is possible for other > architectures to use it.
To clarify, I meant TARGET_SCHED_INIT hook, which is currently not used in the x86 backend (the corresponding function would be ix86_sched_init, not ix86_sched_init_global). Your new implementation is not efficient: when looping over BBs, you need to look only at the last insn of each basic block. Alexander
