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

A. Skrobov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 22 08:16:43 PDT 2017

tyomitch added a comment.

> This needs a much better comment explaining why this is necessary


> I spent a little time experimenting with multiclassing the Thumb1 instructions to avoid optional defs;
>  it seems like it's viable, and it's substantially simpler in some ways, but it would take a lot of time to finish.

Yes, that's my own understanding, too.


More information about the llvm-commits mailing list