[PATCH] D25140: ScheduleDAGInstrs: Add condjump deps in addSchedBarrierDeps()

[Matthias Braun via llvm-commits]

MatzeB added a comment.

In https://reviews.llvm.org/D25140#559480, @kparzysz wrote:

> Should def operands in branches also be treated specially (e.g. adding a dependency with a successor's live-in)?


It shouldn't matter for scheduling: Branch instructions are only allowed at the end of a basic block, so by definition we cannot have any non-branch instructions between them, so we will never get any interesting scheduling region formed between branches.


Repository:
  rL LLVM

https://reviews.llvm.org/D25140