[PATCH] D31081: [ARM] ScheduleDAGRRList::DelayForLiveRegsBottomUp must consider OptionalDefs

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

Unfortunately, I'm not that familiar with ScheduleDAG, so I'm not sure I can give good guidance.  I see two things very wrong with this patch, though.  One, it prevents schedules which are perfectly legal for non-Thumb1 code.  Two, you're specifically updating one of the schedulers, but they're all affected by this issue.


https://reviews.llvm.org/D31081