add_branch_dependences in sched-rgn.c

[Kyrill Tkachov <kyrylo dot tkachov at arm dot com>]

Hi all,

I'm looking at some curious pre-reload scheduling behaviour and I noticed this:

At the add_branch_dependences function sched-rgn.c there is a comment that says 
"branches, calls, uses, clobbers, cc0 setters, and instructions that can throw 
exceptions" should be scheduled at the end of the basic block.

However right below it the code that detects this kind of insns seems to only 
look for these insns that are directly adjacent to the end of the block 
(implemented with a while loop that ends as soon as the current insn is not one 
of the aforementioned).

Shouldn't the code look through the whole basic block, gather all of the 
branches, clobbers etc. and schedule them at the end?


Any ideas?

Thanks,
Kyrill