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

[A. Skrobov via Phabricator via llvm-commits]

tyomitch added a comment.

> One, it prevents schedules which are perfectly legal for non-Thumb1 code.

No, it doesn't: for the test case that I'm adding, the Thumb2 output remains a sequence of (adds, eor, eor, adc).
This is because the code I'm adding checks not only the presence but also the actual value of the OptionalDef operand, and its value is `%noreg` when the instruction doesn't set CPSR.


Repository:
  rL LLVM

https://reviews.llvm.org/D31081