[PATCH] D127971: [MachinePipeliner] Handle failing constrainRegClass

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jun 19 10:25:12 PDT 2022


dmgreen added a comment.

In D127971#3590721 <https://reviews.llvm.org/D127971#3590721>, @bcahoon wrote:

> Thanks for making this change. It's an interesting case. We handled a related issue, with subregisters, by adding code to preprocessPhiNodes to create copies. I don't have a problem with approach in this patch. There may be other cases in this code that need a similar change? Not sure...

Thanks for the suggestions. I've looked through and checked the other uses of constrainRegClass, changing them to asserts so that it is more obvious where the call fails.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D127971/new/

https://reviews.llvm.org/D127971



More information about the llvm-commits mailing list